From: Andrew Burgess <aburgess@redhat.com>
Date: Tue, 30 Apr 2024 14:48:21 +0000 (+0100)
Subject: gdb: convert address_in_mem_range to mem_range::contains
X-Git-Tag: gdb-15-branchpoint~145
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=cba95c27876724059c3e99ea1857fb19b9cf8220;p=thirdparty%2Fbinutils-gdb.git

gdb: convert address_in_mem_range to mem_range::contains

Replace the global function address_in_mem_range with the member
function mem_range::contains.  The implementation of the function
doesn't change.

There should be no user visible changes after this commit.

Approved-By: Tom Tromey <tom@tromey.com>
---

diff --git a/gdb/corelow.c b/gdb/corelow.c
index a710b07f968..462a05591c1 100644
--- a/gdb/corelow.c
+++ b/gdb/corelow.c
@@ -994,9 +994,9 @@ core_target::xfer_memory_via_mappings (gdb_byte *readbuf,
 
   for (const auto &mr : m_core_unavailable_mappings)
     {
-      if (address_in_mem_range (memaddr, &mr))
+      if (mr.contains (memaddr))
 	{
-	  if (!address_in_mem_range (memend, &mr))
+	  if (!mr.contains (memend))
 	    len = mr.start + mr.length - memaddr;
 
 	  xfer_status = this->beneath ()->xfer_partial (TARGET_OBJECT_MEMORY,
diff --git a/gdb/memrange.c b/gdb/memrange.c
index 11687b808b8..58749b757ca 100644
--- a/gdb/memrange.c
+++ b/gdb/memrange.c
@@ -33,11 +33,11 @@ mem_ranges_overlap (CORE_ADDR start1, int len1,
 
 /* See memrange.h.  */
 
-int
-address_in_mem_range (CORE_ADDR address, const struct mem_range *r)
+bool
+mem_range::contains (CORE_ADDR address) const
 {
-  return (r->start <= address
-	  && (address - r->start) < r->length);
+  return (this->start <= address
+	  && (address - this->start) < this->length);
 }
 
 void
diff --git a/gdb/memrange.h b/gdb/memrange.h
index 74df82588f4..d83fcbd57a6 100644
--- a/gdb/memrange.h
+++ b/gdb/memrange.h
@@ -41,6 +41,9 @@ struct mem_range
 	    && this->length == other.length);
   }
 
+  /* Returns true if ADDR is in this mem_range.  */
+  bool contains (CORE_ADDR addr) const;
+
   /* Lowest address in the range.  */
   CORE_ADDR start;
 
@@ -54,11 +57,6 @@ struct mem_range
 extern int mem_ranges_overlap (CORE_ADDR start1, int len1,
 			       CORE_ADDR start2, int len2);
 
-/* Returns true if ADDR is in RANGE.  */
-
-extern int address_in_mem_range (CORE_ADDR addr,
-				 const struct mem_range *range);
-
 /* Sort ranges by start address, then coalesce contiguous or
    overlapping ranges.  */
 
diff --git a/gdb/solib-svr4.c b/gdb/solib-svr4.c
index 1dd04c29c98..1d4a50568d7 100644
--- a/gdb/solib-svr4.c
+++ b/gdb/solib-svr4.c
@@ -1475,7 +1475,7 @@ svr4_current_sos ()
 
 	  auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so->lm_info.get ());
 
-	  if (address_in_mem_range (li->l_ld, &vsyscall_range))
+	  if (vsyscall_range.contains (li->l_ld))
 	    {
 	      auto next = sos.erase (so);
 	      delete &*so;