diff options
author | Andrew Burgess <aburgess@redhat.com> | 2024-04-30 15:48:21 +0100 |
---|---|---|
committer | Andrew Burgess <aburgess@redhat.com> | 2024-05-09 10:10:50 +0100 |
commit | cba95c27876724059c3e99ea1857fb19b9cf8220 (patch) | |
tree | a8cf28cd5e5347b3c2179c2bd6a2295409e05bfb | |
parent | cb1a6b85b84359419e7fcf9069c3ba9d27a856c7 (diff) | |
download | gdb-cba95c27876724059c3e99ea1857fb19b9cf8220.zip gdb-cba95c27876724059c3e99ea1857fb19b9cf8220.tar.gz gdb-cba95c27876724059c3e99ea1857fb19b9cf8220.tar.bz2 |
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>
-rw-r--r-- | gdb/corelow.c | 4 | ||||
-rw-r--r-- | gdb/memrange.c | 8 | ||||
-rw-r--r-- | gdb/memrange.h | 8 | ||||
-rw-r--r-- | gdb/solib-svr4.c | 2 |
4 files changed, 10 insertions, 12 deletions
diff --git a/gdb/corelow.c b/gdb/corelow.c index a710b07f..462a055 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 11687b8..58749b7 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 74df825..d83fcbd 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 1dd04c2..1d4a505 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; |