From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id YDyUARYX0mAtYAAAWB0awg (envelope-from ) for ; Tue, 22 Jun 2021 13:00:06 -0400 Received: by simark.ca (Postfix, from userid 112) id 03A3F1F1F2; Tue, 22 Jun 2021 13:00:05 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-1.1 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id C93311E54D for ; Tue, 22 Jun 2021 13:00:01 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 82ECF3945065 for ; Tue, 22 Jun 2021 17:00:01 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 82ECF3945065 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1624381201; bh=GvViGRuOyJ1RsqQDubgqj0ip1q7KktqrkbMkeK/mmU8=; h=To:Subject:Date:In-Reply-To:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=paISO5wFsTxzOOPHRkboTiW18Vq7kezr5OIetW3qIZKMRwRAhEfRtCgJvEtYB97je cxKb1WTWvvKUQk9nDuZoC0EljuvhN4IIGv0+auZ4xjjNNLpaQRAVaBjfhzeAD7PBn1 3nOhJGZ9aA4PrSsVqezfgKdG8hiv67GxoZw5ogT0= Received: from barracuda.ebox.ca (barracuda.ebox.ca [96.127.255.19]) by sourceware.org (Postfix) with ESMTPS id CA3C83857426 for ; Tue, 22 Jun 2021 16:57:10 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org CA3C83857426 X-ASG-Debug-ID: 1624381026-0c856e67e2169e530001-fS2M51 Received: from smtp.ebox.ca (smtp.ebox.ca [96.127.255.82]) by barracuda.ebox.ca with ESMTP id MU383XNuQMX3qK5O (version=TLSv1 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Tue, 22 Jun 2021 12:57:06 -0400 (EDT) X-Barracuda-Envelope-From: simon.marchi@polymtl.ca X-Barracuda-RBL-Trusted-Forwarder: 96.127.255.82 Received: from simark.localdomain (192-222-157-6.qc.cable.ebox.net [192.222.157.6]) by smtp.ebox.ca (Postfix) with ESMTP id 267C9441D64; Tue, 22 Jun 2021 12:57:06 -0400 (EDT) X-Barracuda-RBL-IP: 192.222.157.6 X-Barracuda-Effective-Source-IP: 192-222-157-6.qc.cable.ebox.net[192.222.157.6] X-Barracuda-Apparent-Source-IP: 192.222.157.6 To: gdb-patches@sourceware.org Subject: [PATCH 01/11] gdb: introduce iterator_range, remove next_adapter Date: Tue, 22 Jun 2021 12:56:54 -0400 X-ASG-Orig-Subj: [PATCH 01/11] gdb: introduce iterator_range, remove next_adapter Message-Id: <20210622165704.2404007-2-simon.marchi@polymtl.ca> X-Mailer: git-send-email 2.32.0 In-Reply-To: <20210622165704.2404007-1-simon.marchi@polymtl.ca> References: <20210622165704.2404007-1-simon.marchi@polymtl.ca> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Barracuda-Connect: smtp.ebox.ca[96.127.255.82] X-Barracuda-Start-Time: 1624381026 X-Barracuda-Encrypted: DHE-RSA-AES256-SHA X-Barracuda-URL: https://96.127.255.19:443/cgi-mod/mark.cgi X-Virus-Scanned: by bsmtpd at ebox.ca X-Barracuda-Scan-Msg-Size: 24635 X-Barracuda-BRTS-Status: 1 X-Barracuda-Spam-Score: 0.50 X-Barracuda-Spam-Status: No, SCORE=0.50 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=8.0 tests=BSF_RULE7568M X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.90826 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- 0.50 BSF_RULE7568M Custom Rule 7568M X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Simon Marchi via Gdb-patches Reply-To: Simon Marchi Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" I was always a bit confused by next_adapter, because it kind of mixes the element type and the iterator type. In reality, it is not much more than a class that wraps two iterators (begin and end). However, it assumes that: - you can construct the begin iterator by passing a pointer to the first element of the iterable - you can default-construct iterator to make the end iterator I think that by generalizing it a little bit, we can re-use it at more places. Rename it to "iterator_range". I think it describes a bit better: it's a range made by wrapping a begin and end iterator. Move it to its own file, since it's not related to next_iterator anymore. iterator_range has two constructors. The variadic one, where arguments are forwarded to construct the underlying begin iterator. The end iterator is constructed through default construction. This is a generalization of what we have today. There is another constructor which receives already constructed begin and end iterators, useful if the end iterator can't be obtained by default-construction. Or, if you wanted to make a range that does not end at the end of the container, you could pass any iterator as the "end". This generalization allows removing some "range" classes, like all_inferiors_range. These classes existed only to pass some arguments when constructing the begin iterator. With iterator_range, those same arguments are passed to the iterator_range constructed and then forwarded to the constructed begin iterator. There is a small functional difference in how iterator_range works compared to next_adapter. next_adapter stored the pointer it received as argument and constructeur an iterator in the `begin` method. iterator_range constructs the begin iterator and stores it as a member. Its `begin` method returns a copy of that iterator. With just iterator_range, uses of next_adapter would be replaced with: using foo_iterator = next_iterator; using foo_range = iterator_range; However, I added a `next_range` wrapper as a direct replacement for next_adapter. IMO, next_range is a slightly better name than next_adapter. The rest of the changes are applications of this new class. gdbsupport/ChangeLog: * next-iterator.h (class next_adapter): Remove. * iterator-range.h: New. gdb/ChangeLog: * breakpoint.h (bp_locations_range): Remove. (bp_location_range): New. (struct breakpoint) : Adjust type. (breakpoint_range): Use iterator_range. (tracepoint_range): Use iterator_range. * breakpoint.c (breakpoint::locations): Adjust return type. * gdb_bfd.h (gdb_bfd_section_range): Use iterator_range. * gdbthread.h (all_threads_safe): Pass argument to all_threads_safe_range. * inferior-iter.h (all_inferiors_range): Use iterator_range. (all_inferiors_safe_range): Use iterator_range. (all_non_exited_inferiors_range): Use iterator_range. * inferior.h (all_inferiors, all_non_exited_inferiors): Pass inferior_list as argument. * objfiles.h (struct objfile) : Remove. : Return compunit_symtab_range. * progspace.h (unwrapping_objfile_iterator) : Take parameter by value. (unwrapping_objfile_range): Use iterator_range. (struct program_space) : Define with "using". : Adjust. : Define with "using". : Adjust. : Return so_list_range, define here. * progspace.c (program_space::solibs): Remove. * psymtab.h (class psymtab_storage) : New. : Use iterator_range. * solist.h (so_list_range): New. * symtab.h (compunit_symtab_range): New. (symtab_range): New. (compunit_filetabs): Change to a function. * thread-iter.h (inf_threads_range, inf_non_exited_threads_range, safe_inf_threads_range, all_threads_safe_range): Use iterator_range. * top.h (ui_range): New. (all_uis): Use ui_range. Change-Id: Ib7a9d2a3547f45f01aa1c6b24536ba159db9b854 --- gdb/breakpoint.c | 4 +-- gdb/breakpoint.h | 10 +++--- gdb/gdb_bfd.h | 4 +-- gdb/gdbthread.h | 2 +- gdb/inferior-iter.h | 69 +++++-------------------------------- gdb/inferior.h | 4 +-- gdb/objfiles.h | 6 ++-- gdb/progspace.c | 8 ----- gdb/progspace.h | 45 ++++++++---------------- gdb/psymtab.h | 2 +- gdb/solist.h | 2 ++ gdb/symtab.h | 15 ++++---- gdb/thread-iter.h | 27 +++------------ gdb/top.h | 6 ++-- gdbsupport/iterator-range.h | 60 ++++++++++++++++++++++++++++++++ gdbsupport/next-iterator.h | 32 +++-------------- 16 files changed, 124 insertions(+), 172 deletions(-) create mode 100644 gdbsupport/iterator-range.h diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 0595c6f8cbd4..7d7e299ad5ff 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -12282,9 +12282,9 @@ breakpoint::~breakpoint () /* See breakpoint.h. */ -bp_locations_range breakpoint::locations () +bp_location_range breakpoint::locations () { - return bp_locations_range (this->loc); + return bp_location_range (this->loc); } static struct bp_location * diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index e40504f14ed3..fe68730f1651 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -30,6 +30,8 @@ #include "gdbsupport/array-view.h" #include "gdbsupport/filtered-iterator.h" #include "gdbsupport/function-view.h" +#include "gdbsupport/next-iterator.h" +#include "gdbsupport/iterator-range.h" #include "gdbsupport/refcounted-object.h" #include "gdbsupport/safe-iterator.h" #include "cli/cli-script.h" @@ -706,7 +708,7 @@ extern bool target_exact_watchpoints; /* bp_location linked list range. */ -using bp_locations_range = next_adapter; +using bp_location_range = next_range; /* Note that the ->silent field is not currently used by any commands (though the code is in there if it was to be, and set_raw_breakpoint @@ -721,7 +723,7 @@ struct breakpoint virtual ~breakpoint (); /* Return a range of this breakpoint's locations. */ - bp_locations_range locations (); + bp_location_range locations (); /* Methods associated with this breakpoint. */ const breakpoint_ops *ops = NULL; @@ -1715,7 +1717,7 @@ using breakpoint_iterator = next_iterator; /* Breakpoint linked list range. */ -using breakpoint_range = next_adapter; +using breakpoint_range = iterator_range; /* Return a range to iterate over all breakpoints. */ @@ -1746,7 +1748,7 @@ using tracepoint_iterator /* Breakpoint linked list range, filtering to only keep tracepoints. */ -using tracepoint_range = next_adapter; +using tracepoint_range = iterator_range; /* Return a range to iterate over all tracepoints. */ diff --git a/gdb/gdb_bfd.h b/gdb/gdb_bfd.h index d366fc0b1195..156c2760f11c 100644 --- a/gdb/gdb_bfd.h +++ b/gdb/gdb_bfd.h @@ -23,6 +23,7 @@ #include "registry.h" #include "gdbsupport/byte-vector.h" #include "gdbsupport/gdb_ref_ptr.h" +#include "gdbsupport/iterator-range.h" #include "gdbsupport/next-iterator.h" DECLARE_REGISTRY (bfd); @@ -208,8 +209,7 @@ gdb_bfd_ref_ptr gdb_bfd_open_from_target_memory (CORE_ADDR addr, ULONGEST size, ... use SECT ... */ -using gdb_bfd_section_iterator = next_iterator; -using gdb_bfd_section_range = next_adapter; +using gdb_bfd_section_range = next_range; static inline gdb_bfd_section_range gdb_bfd_sections (bfd *abfd) diff --git a/gdb/gdbthread.h b/gdb/gdbthread.h index eef37f79e6ad..f19c88f9bb4a 100644 --- a/gdb/gdbthread.h +++ b/gdb/gdbthread.h @@ -567,7 +567,7 @@ all_non_exited_threads (process_stratum_target *proc_target = nullptr, inline all_threads_safe_range all_threads_safe () { - return {}; + return all_threads_safe_range (all_threads_iterator::begin_t {}); } extern int thread_count (process_stratum_target *proc_target); diff --git a/gdb/inferior-iter.h b/gdb/inferior-iter.h index 2ae0a0957d93..f999150a7b55 100644 --- a/gdb/inferior-iter.h +++ b/gdb/inferior-iter.h @@ -90,6 +90,11 @@ class all_inferiors_iterator inferior *m_inf; }; +/* A range adapter that makes it possible to iterate over all + inferiors with range-for. */ + +using all_inferiors_range = iterator_range; + /* Filter for filtered_iterator. Filters out exited inferiors. */ struct exited_inferior_filter @@ -106,21 +111,10 @@ using all_non_exited_inferiors_iterator = filtered_iterator; /* A range adapter that makes it possible to iterate over all - inferiors with range-for. */ -struct all_inferiors_range -{ - all_inferiors_range (process_stratum_target *proc_target = nullptr) - : m_filter_target (proc_target) - {} + non-exited inferiors with range-for. */ - all_inferiors_iterator begin () const - { return all_inferiors_iterator (m_filter_target, inferior_list); } - all_inferiors_iterator end () const - { return all_inferiors_iterator (); } - -private: - process_stratum_target *m_filter_target; -}; +using all_non_exited_inferiors_range + = iterator_range; /* Iterate over all inferiors, safely. */ @@ -131,51 +125,6 @@ using all_inferiors_safe_iterator inferiors with range-for "safely". I.e., it is safe to delete the currently-iterated inferior. */ -struct all_inferiors_safe_range -{ - explicit all_inferiors_safe_range (process_stratum_target *filter_target) - : m_filter_target (filter_target) - {} - - all_inferiors_safe_range () - : m_filter_target (nullptr) - {} - - all_inferiors_safe_iterator begin () const - { - return (all_inferiors_safe_iterator - (all_inferiors_iterator (m_filter_target, inferior_list))); - } - - all_inferiors_safe_iterator end () const - { return all_inferiors_safe_iterator (); } - -private: - /* The filter. */ - process_stratum_target *m_filter_target; -}; - -/* A range adapter that makes it possible to iterate over all - non-exited inferiors with range-for. */ - -struct all_non_exited_inferiors_range -{ - explicit all_non_exited_inferiors_range (process_stratum_target *filter_target) - : m_filter_target (filter_target) - {} - - all_non_exited_inferiors_range () - : m_filter_target (nullptr) - {} - - all_non_exited_inferiors_iterator begin () const - { return all_non_exited_inferiors_iterator (m_filter_target, inferior_list); } - all_non_exited_inferiors_iterator end () const - { return all_non_exited_inferiors_iterator (); } - -private: - /* The filter. */ - process_stratum_target *m_filter_target; -}; +using all_inferiors_safe_range = iterator_range; #endif /* !defined (INFERIOR_ITER_H) */ diff --git a/gdb/inferior.h b/gdb/inferior.h index f61b5889e858..c63990aabe0e 100644 --- a/gdb/inferior.h +++ b/gdb/inferior.h @@ -681,7 +681,7 @@ all_inferiors_safe () inline all_inferiors_range all_inferiors (process_stratum_target *proc_target = nullptr) { - return all_inferiors_range (proc_target); + return all_inferiors_range (proc_target, inferior_list); } /* Return a range that can be used to walk over all inferiors with PID @@ -690,7 +690,7 @@ all_inferiors (process_stratum_target *proc_target = nullptr) inline all_non_exited_inferiors_range all_non_exited_inferiors (process_stratum_target *proc_target = nullptr) { - return all_non_exited_inferiors_range (proc_target); + return all_non_exited_inferiors_range (proc_target, inferior_list); } /* Prune away automatically added inferiors that aren't required diff --git a/gdb/objfiles.h b/gdb/objfiles.h index 5a8a782a6462..91557b89f6b5 100644 --- a/gdb/objfiles.h +++ b/gdb/objfiles.h @@ -476,14 +476,12 @@ struct objfile DISABLE_COPY_AND_ASSIGN (objfile); - typedef next_adapter compunits_range; - /* A range adapter that makes it possible to iterate over all compunits in one objfile. */ - compunits_range compunits () + compunit_symtab_range compunits () { - return compunits_range (compunit_symtabs); + return compunit_symtab_range (compunit_symtabs); } /* A range adapter that makes it possible to iterate over all diff --git a/gdb/progspace.c b/gdb/progspace.c index ebbc784dcfbb..e3cc6929a236 100644 --- a/gdb/progspace.c +++ b/gdb/progspace.c @@ -215,14 +215,6 @@ program_space::remove_objfile (struct objfile *objfile) /* See progspace.h. */ -next_adapter -program_space::solibs () const -{ - return next_adapter (this->so_list); -} - -/* See progspace.h. */ - void program_space::exec_close () { diff --git a/gdb/progspace.h b/gdb/progspace.h index 790684743d87..fb348ca7539a 100644 --- a/gdb/progspace.h +++ b/gdb/progspace.h @@ -25,6 +25,7 @@ #include "gdb_bfd.h" #include "gdbsupport/gdb_vecs.h" #include "registry.h" +#include "solist.h" #include "gdbsupport/next-iterator.h" #include "gdbsupport/safe-iterator.h" #include @@ -59,8 +60,8 @@ class unwrapping_objfile_iterator typedef typename objfile_list::iterator::iterator_category iterator_category; typedef typename objfile_list::iterator::difference_type difference_type; - unwrapping_objfile_iterator (const objfile_list::iterator &iter) - : m_iter (iter) + unwrapping_objfile_iterator (objfile_list::iterator iter) + : m_iter (std::move (iter)) { } @@ -89,29 +90,7 @@ class unwrapping_objfile_iterator /* A range that returns unwrapping_objfile_iterators. */ -struct unwrapping_objfile_range -{ - typedef unwrapping_objfile_iterator iterator; - - unwrapping_objfile_range (objfile_list &ol) - : m_list (ol) - { - } - - iterator begin () const - { - return iterator (m_list.begin ()); - } - - iterator end () const - { - return iterator (m_list.end ()); - } - -private: - - objfile_list &m_list; -}; +using unwrapping_objfile_range = iterator_range; /* A program space represents a symbolic view of an address space. Roughly speaking, it holds all the data associated with a @@ -222,7 +201,7 @@ struct program_space a program space. */ ~program_space (); - typedef unwrapping_objfile_range objfiles_range; + using objfiles_range = unwrapping_objfile_range; /* Return an iterable object that can be used to iterate over all objfiles. The basic use is in a foreach, like: @@ -230,10 +209,12 @@ struct program_space for (objfile *objf : pspace->objfiles ()) { ... } */ objfiles_range objfiles () { - return unwrapping_objfile_range (objfiles_list); + return objfiles_range + (unwrapping_objfile_iterator (objfiles_list.begin ()), + unwrapping_objfile_iterator (objfiles_list.end ())); } - typedef basic_safe_range objfiles_safe_range; + using objfiles_safe_range = basic_safe_range; /* An iterable object that can be used to iterate over all objfiles. The basic use is in a foreach, like: @@ -244,7 +225,10 @@ struct program_space deleted during iteration. */ objfiles_safe_range objfiles_safe () { - return objfiles_safe_range (objfiles_list); + return objfiles_safe_range + (objfiles_range + (unwrapping_objfile_iterator (objfiles_list.begin ()), + unwrapping_objfile_iterator (objfiles_list.end ()))); } /* Add OBJFILE to the list of objfiles, putting it just before @@ -270,7 +254,8 @@ struct program_space program space. Use it like: for (so_list *so : pspace->solibs ()) { ... } */ - next_adapter solibs () const; + so_list_range solibs () const + { return so_list_range (this->so_list); } /* Close and clear exec_bfd. If we end up with no target sections to read memory from, this unpushes the exec_ops target. */ diff --git a/gdb/psymtab.h b/gdb/psymtab.h index 522ccf3a12a4..7cd3e95b179b 100644 --- a/gdb/psymtab.h +++ b/gdb/psymtab.h @@ -104,7 +104,7 @@ class psymtab_storage void install_psymtab (partial_symtab *pst); - typedef next_adapter partial_symtab_range; + using partial_symtab_range = next_range; /* A range adapter that makes it possible to iterate over all psymtabs in one objfile. */ diff --git a/gdb/solist.h b/gdb/solist.h index d44e6f4048cc..6033751dd695 100644 --- a/gdb/solist.h +++ b/gdb/solist.h @@ -163,6 +163,8 @@ struct target_so_ops void (*handle_event) (void); }; +using so_list_range = next_range; + /* Free the memory associated with a (so_list *). */ void free_so (struct so_list *so); diff --git a/gdb/symtab.h b/gdb/symtab.h index a5d0168faf08..759cf4d772c3 100644 --- a/gdb/symtab.h +++ b/gdb/symtab.h @@ -33,6 +33,7 @@ #include "gdbsupport/gdb_optional.h" #include "gdbsupport/gdb_string_view.h" #include "gdbsupport/next-iterator.h" +#include "gdbsupport/iterator-range.h" #include "completer.h" #include "gdb-demangle.h" @@ -1521,6 +1522,8 @@ struct compunit_symtab struct compunit_symtab *user; }; +using compunit_symtab_range = next_range; + #define COMPUNIT_OBJFILE(cust) ((cust)->objfile) #define COMPUNIT_FILETABS(cust) ((cust)->filetabs) #define COMPUNIT_DEBUGFORMAT(cust) ((cust)->debugformat) @@ -1536,13 +1539,13 @@ struct compunit_symtab /* A range adapter to allowing iterating over all the file tables within a compunit. */ -struct compunit_filetabs : public next_adapter +using symtab_range = next_range; + +static inline symtab_range +compunit_filetabs (compunit_symtab *cu) { - compunit_filetabs (struct compunit_symtab *cu) - : next_adapter (cu->filetabs) - { - } -}; + return symtab_range (cu->filetabs); +} /* Return the primary symtab of CUST. */ diff --git a/gdb/thread-iter.h b/gdb/thread-iter.h index 853098620e8e..098af0f3241b 100644 --- a/gdb/thread-iter.h +++ b/gdb/thread-iter.h @@ -170,43 +170,24 @@ using safe_inf_threads_iterator /* A range adapter that makes it possible to iterate over all threads of an inferior with range-for. */ -using inf_threads_range - = next_adapter; +using inf_threads_range = iterator_range; /* A range adapter that makes it possible to iterate over all non-exited threads of an inferior with range-for. */ using inf_non_exited_threads_range - = next_adapter; + = iterator_range; /* A range adapter that makes it possible to iterate over all threads of an inferior with range-for, safely. */ -using safe_inf_threads_range - = next_adapter; - -/* A range adapter that makes it possible to iterate over all threads - of all inferiors with range-for. */ - -struct all_threads_range -{ - all_threads_iterator begin () const - { return all_threads_iterator (all_threads_iterator::begin_t {}); } - all_threads_iterator end () const - { return all_threads_iterator (); } -}; +using safe_inf_threads_range = iterator_range; /* A range adapter that makes it possible to iterate over all threads with range-for "safely". I.e., it is safe to delete the currently-iterated thread. */ -struct all_threads_safe_range -{ - all_threads_safe_iterator begin () const - { return all_threads_safe_iterator (all_threads_iterator::begin_t {}); } - all_threads_safe_iterator end () const - { return all_threads_safe_iterator (); } -}; +using all_threads_safe_range = iterator_range; /* A range adapter that makes it possible to iterate over all threads that match a PTID filter with range-for. */ diff --git a/gdb/top.h b/gdb/top.h index 56bd06c698be..ef88ca024e4e 100644 --- a/gdb/top.h +++ b/gdb/top.h @@ -204,11 +204,13 @@ class switch_thru_all_uis #define SWITCH_THRU_ALL_UIS() \ for (switch_thru_all_uis stau_state; !stau_state.done (); stau_state.next ()) +using ui_range = next_range; + /* An adapter that can be used to traverse over all UIs. */ static inline -next_adapter all_uis () +ui_range all_uis () { - return next_adapter (ui_list); + return ui_range (ui_list); } /* Register the UI's input file descriptor in the event loop. */ diff --git a/gdbsupport/iterator-range.h b/gdbsupport/iterator-range.h new file mode 100644 index 000000000000..2a718e6c83a1 --- /dev/null +++ b/gdbsupport/iterator-range.h @@ -0,0 +1,60 @@ +/* A range adapter that wraps begin / end iterators. + Copyright (C) 2021 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +#ifndef GDBSUPPORT_ITERATOR_RANGE_H +#define GDBSUPPORT_ITERATOR_RANGE_H + +/* A wrapper that allows using ranged for-loops on a range described by two + iterators. */ + +template +struct iterator_range +{ + using iterator = IteratorType; + + /* Create an iterator_range using BEGIN as the begin iterator. + + Assume that the end iterator can be default-constructed. */ + template + iterator_range (Args &&...args) + : m_begin (std::forward (args)...) + {} + + /* Create an iterator range using explicit BEGIN and END iterators. */ + template + iterator_range (IteratorType begin, IteratorType end) + : m_begin (std::move (begin)), m_end (std::move (end)) + {} + + /* Need these as the variadic constructor would be a better match + otherwise. */ + iterator_range (iterator_range &) = default; + iterator_range (const iterator_range &) = default; + iterator_range (iterator_range &&) = default; + + IteratorType begin () const + { return m_begin; } + + IteratorType end () const + { return m_end; } + +private: + IteratorType m_begin, m_end; +}; + +#endif /* GDBSUPPORT_ITERATOR_RANGE_H */ diff --git a/gdbsupport/next-iterator.h b/gdbsupport/next-iterator.h index 501752499846..72b241ea2b71 100644 --- a/gdbsupport/next-iterator.h +++ b/gdbsupport/next-iterator.h @@ -19,6 +19,8 @@ #ifndef COMMON_NEXT_ITERATOR_H #define COMMON_NEXT_ITERATOR_H +#include "gdbsupport/iterator-range.h" + /* An iterator that uses the 'next' field of a type to iterate. This can be used with various GDB types that are stored as linked lists. */ @@ -70,33 +72,9 @@ struct next_iterator T *m_item; }; -/* A range adapter that allows iterating over a linked list. */ - -template> -class next_adapter -{ -public: - - explicit next_adapter (T *item) - : m_item (item) - { - } - - using iterator = Iterator; - - iterator begin () const - { - return iterator (m_item); - } - - iterator end () const - { - return iterator (); - } +/* A convenience wrapper to make a range type around a next_iterator. */ -private: - - T *m_item; -}; +template +using next_range = iterator_range>; #endif /* COMMON_NEXT_ITERATOR_H */ -- 2.32.0