From: Tom de Vries Date: Tue, 16 Dec 2025 15:20:22 +0000 (+0100) Subject: [gdb] Fix "the the" X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6cf6a77c2b0293b9560d982436cda8856f082c93;p=thirdparty%2Fbinutils-gdb.git [gdb] Fix "the the" Replace "the the" with "the". Result of running: ... $ find gdb* -type f | egrep -v ChangeLog | xargs sed -i 's/the the /the /' $ find gdb* -type f | egrep -v ChangeLog | xargs sed -i 's/the the$/the/' ... The only change I have doubts about, is this comment in queue_comp_unit in gdb/dwarf2/read.c: ... ... . If the CU gets enqueued by this function but its DIEs - are not yet loaded, the the caller must load the CU's DIEs to ensure the + are not yet loaded, the caller must load the CU's DIEs to ensure the invariant is respected. ... where I think "the the" -> "then the" also make sense. But for now, I'm going with "the the" -> "the". Tested by building gdb on x86_64-linux. Approved-By: Simon Marchi --- diff --git a/gdb/README b/gdb/README index 5dc2e7e7ad3..492b0891668 100644 --- a/gdb/README +++ b/gdb/README @@ -421,7 +421,7 @@ more obscure GDB `configure' options are not listed here. `--enable-binary-file-formats=all' Configure GDB to only be be able to read selected file formats. The special value "all" causes all file formats to be compiled - in, and is the the default behavior of the option. This option + in, and is the default behavior of the option. This option is meant for advanced users who are sure of what they expect, if you are unsure which options you will need on your debugging sessions, we recommend that you not use this feature. The diff --git a/gdb/aarch64-linux-nat.c b/gdb/aarch64-linux-nat.c index b4bc921f767..22fb3361c0b 100644 --- a/gdb/aarch64-linux-nat.c +++ b/gdb/aarch64-linux-nat.c @@ -767,7 +767,7 @@ aarch64_linux_nat_target::fetch_registers (struct regcache *regcache, } /* The AArch64 version of the "store_registers" target_ops method. Copy - the value of register REGNO from REGCACHE into the the target. */ + the value of register REGNO from REGCACHE into the target. */ static void aarch64_store_registers (struct regcache *regcache, int regno) @@ -844,7 +844,7 @@ aarch64_store_registers (struct regcache *regcache, int regno) /* A version of the "store_registers" target_ops method used when running 32-bit ARM code on an AArch64 target. Copy the value of register REGNO - from REGCACHE into the the target. */ + from REGCACHE into the target. */ static void aarch32_store_registers (struct regcache *regcache, int regno) diff --git a/gdb/ada-lex.l b/gdb/ada-lex.l index acd8cf3690b..178b53b3236 100644 --- a/gdb/ada-lex.l +++ b/gdb/ada-lex.l @@ -375,7 +375,7 @@ processInt (struct parser_state *par_state, const char *base0, long exp; int base; /* For the based literal with an "f" prefix, we'll return a - floating-point number. This counts the the number of "l"s seen, + floating-point number. This counts the number of "l"s seen, to decide the width of the floating-point number to return. -1 means no "f". */ int floating_point_l_count = -1; diff --git a/gdb/amdgpu-tdep.c b/gdb/amdgpu-tdep.c index ec2fac09448..1cbb1cd6b19 100644 --- a/gdb/amdgpu-tdep.c +++ b/gdb/amdgpu-tdep.c @@ -84,7 +84,7 @@ amdgpu_register_name (struct gdbarch *gdbarch, int regnum) However, at most one register with a given name is actually allocated for a specific wave. If INFERIOR_PTID represents a GPU wave, we query amd-dbgapi to know whether the requested register actually exists for the - current wave, so there won't be duplicates in the the register names we + current wave, so there won't be duplicates in the register names we report for that wave. But there are two known cases where INFERIOR_PTID doesn't represent a GPU diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index eef985bb276..f98d766dbf7 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -3449,7 +3449,7 @@ arm_m_exception_cache (const frame_info_ptr &this_frame) to the exception and if FPU is used (causing extended stack frame). */ /* In the lockup state PC contains a lockup magic value. - The PC value of the the next outer frame is irreversibly + The PC value of the next outer frame is irreversibly lost. The other registers are intact so LR likely contains PC of some frame next to the outer one, but we cannot analyze the next outer frame without knowing its PC diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index f77fe17c81f..61e43d971fb 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -8401,7 +8401,7 @@ default, ten instructions are disassembled. This can be changed using the @code{set record instruction-history-size} command. Instructions are printed in execution order. -It can also print mixed source+disassembly if you specify the the +It can also print mixed source+disassembly if you specify the @code{/m} or @code{/s} modifier, and print the raw instructions in hex as well as in symbolic form by specifying the @code{/r} or @code{/b} modifier. The behavior of the @code{/m}, @code{/s}, @code{/r}, and diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index 162c0bd57bb..12f3e5f0e3d 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -4071,7 +4071,7 @@ queue_comp_unit (dwarf2_per_cu *per_cu, dwarf2_per_objfile *per_objfile) (present in `dwarf2_per_bfd::queue`), then its DIEs are loaded (a dwarf2_cu object exists for this CU, and `dwarf2_per_objfile::get_cu` returns non-nullptr). If the CU gets enqueued by this function but its DIEs - are not yet loaded, the the caller must load the CU's DIEs to ensure the + are not yet loaded, the caller must load the CU's DIEs to ensure the invariant is respected. The caller is therefore not required to load the CU's DIEs (we return false) diff --git a/gdb/gdbarch-gen.h b/gdb/gdbarch-gen.h index 180e7ced7ed..9ae614fe6dd 100644 --- a/gdb/gdbarch-gen.h +++ b/gdb/gdbarch-gen.h @@ -487,7 +487,7 @@ extern void set_gdbarch_return_value_as_value (struct gdbarch *gdbarch, gdbarch_ /* Return the address at which the value being returned from the current function will be stored. This routine is only - called if the current function uses the the "struct return + called if the current function uses the "struct return convention". May return 0 when unable to determine that address. */ diff --git a/gdb/gdbarch_components.py b/gdb/gdbarch_components.py index eac068028ed..b19c32aca27 100644 --- a/gdb/gdbarch_components.py +++ b/gdb/gdbarch_components.py @@ -930,7 +930,7 @@ Function( comment=""" Return the address at which the value being returned from the current function will be stored. This routine is only -called if the current function uses the the "struct return +called if the current function uses the "struct return convention". May return 0 when unable to determine that address.""", diff --git a/gdb/gdbthread.h b/gdb/gdbthread.h index 28bac23cf74..0f6c5b126b6 100644 --- a/gdb/gdbthread.h +++ b/gdb/gdbthread.h @@ -146,7 +146,7 @@ struct thread_control_state any inlined frames). */ struct frame_id step_stack_frame_id {}; - /* True if the the thread is presently stepping over a breakpoint or + /* True if the thread is presently stepping over a breakpoint or a watchpoint, either with an inline step over or a displaced (out of line) step, and we're now expecting it to report a trap for the finished single step. */ @@ -1009,7 +1009,7 @@ extern void global_thread_step_over_chain_remove (thread_info *tp); extern int thread_is_in_step_over_chain (struct thread_info *tp); -/* Return the length of the the step over chain TP is in. +/* Return the length of the step over chain TP is in. If TP is non-nullptr, the thread must be in a step over chain. TP may be nullptr, in which case it denotes an empty list, so a length of diff --git a/gdb/minsyms.c b/gdb/minsyms.c index 90a0c601c59..0ee9e236232 100644 --- a/gdb/minsyms.c +++ b/gdb/minsyms.c @@ -1069,7 +1069,7 @@ const struct gnu_ifunc_fns *gnu_ifunc_fns_p = &stub_gnu_ifunc_fns; /* Return the leading symbol character for BFD ABFD. If ABFD is nullptr, - return the leading symbol character from the the main objfile of PSPACE.. */ + return the leading symbol character from the main objfile of PSPACE.. */ static int get_symbol_leading_char (program_space *pspace, bfd *abfd) diff --git a/gdb/ppc-linux-nat.c b/gdb/ppc-linux-nat.c index fd425654df7..32c8de8cf67 100644 --- a/gdb/ppc-linux-nat.c +++ b/gdb/ppc-linux-nat.c @@ -2793,7 +2793,7 @@ ppc_linux_nat_target::low_new_fork (struct lwp_info *parent, } } -/* Copy the thread debug register state from the PARENT thread to the the +/* Copy the thread debug register state from the PARENT thread to the state for CHILD_LWP, if we're using the HWDEBUG interface. We assume the kernel copies the debug registers from one thread to another after a clone event. The low_prepare_to_resume function is made to work diff --git a/gdb/symtab.c b/gdb/symtab.c index 2eda0da532d..8d313ac0ef5 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -4047,7 +4047,7 @@ find_epilogue_using_linetable (CORE_ADDR func_addr) } gdb_assert (it->unrelocated_pc () < unrel_end); - /* We're at the the last linetable entry of the current function. This + /* We're at the last linetable entry of the current function. This is probably where the epilogue begins, but since the DWARF 5 spec doesn't guarantee it, we iterate backwards through the current function until we either find the epilogue beginning, or are sure diff --git a/gdb/symtab.h b/gdb/symtab.h index 9336f5d35a7..e142040ad56 100644 --- a/gdb/symtab.h +++ b/gdb/symtab.h @@ -736,7 +736,7 @@ struct minimal_symbol : public general_symbol_info return m_value.unrel_addr; } - /* The unrelocated address just after the end of the the minimal + /* The unrelocated address just after the end of the minimal symbol. */ unrelocated_addr unrelocated_end_address () const { diff --git a/gdb/testsuite/gdb.base/corefile3.exp b/gdb/testsuite/gdb.base/corefile3.exp index bee393c983e..0f18dc26726 100644 --- a/gdb/testsuite/gdb.base/corefile3.exp +++ b/gdb/testsuite/gdb.base/corefile3.exp @@ -14,7 +14,7 @@ # along with this program. If not, see . # Create a core file with some mapped file regions, but ensure that -# the the kernel should write the regions into the core file (e.g. r/w +# the kernel should write the regions into the core file (e.g. r/w # file backed mapping). # # We then delete the file that backed the mapping and load the core diff --git a/gdb/testsuite/gdb.base/stap-probe.exp b/gdb/testsuite/gdb.base/stap-probe.exp index 540daf3970b..277a92789cc 100644 --- a/gdb/testsuite/gdb.base/stap-probe.exp +++ b/gdb/testsuite/gdb.base/stap-probe.exp @@ -97,7 +97,7 @@ proc check_for_usable_xmm0_probe { binfile } { verbose -log "APB: Matched on '$ignore'" verbose -log "APB: arguments: '$arguments'" - # Check the the argument string mentions xmm0. + # Check the argument string mentions xmm0. if {![regexp {@%?xmm0} $arguments]} { verbose -log "APB: Prove doesn't use xmm0 register" return false diff --git a/gdb/testsuite/gdb.base/testenv.exp b/gdb/testsuite/gdb.base/testenv.exp index a8fb3206497..a6fe086dc1a 100644 --- a/gdb/testsuite/gdb.base/testenv.exp +++ b/gdb/testsuite/gdb.base/testenv.exp @@ -31,7 +31,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } { return -1 } -# Test that the the inferior sees EXPECTED env vars starting with +# Test that the inferior sees EXPECTED env vars starting with # "TEST_GDB". proc test_num_test_vars {expected message} { set num [get_integer_valueof "j" -1 "$message, get num vars"] diff --git a/gdb/testsuite/gdb.cp/psymtab-parameter.exp b/gdb/testsuite/gdb.cp/psymtab-parameter.exp index cbe51a5ad60..ab30fd389ef 100644 --- a/gdb/testsuite/gdb.cp/psymtab-parameter.exp +++ b/gdb/testsuite/gdb.cp/psymtab-parameter.exp @@ -28,7 +28,7 @@ clean_restart $testfile.x gdb_test_no_output "set language c++" if { ![readnow] } { - # Check that the the CU (Compilation Unit) is unexpanded. + # Check that the CU (Compilation Unit) is unexpanded. gdb_test_no_output "maintenance info symtabs" } diff --git a/gdb/testsuite/gdb.fortran/types.exp b/gdb/testsuite/gdb.fortran/types.exp index 03bc2ce6799..766a1bc843f 100644 --- a/gdb/testsuite/gdb.fortran/types.exp +++ b/gdb/testsuite/gdb.fortran/types.exp @@ -79,7 +79,7 @@ proc test_default_types {} { gdb_test "ptype complex" "type = complex\\*4" } -# Test the the primitive Fortran types, those that GDB should always +# Test the primitive Fortran types, those that GDB should always # know, even if the program does not define them, are in fact, known. proc test_primitive_types_known {} { foreach type {void character \ diff --git a/gdb/testsuite/gdb.python/py-styled-execute.exp b/gdb/testsuite/gdb.python/py-styled-execute.exp index 198dab52ddd..7b7f8326f2a 100644 --- a/gdb/testsuite/gdb.python/py-styled-execute.exp +++ b/gdb/testsuite/gdb.python/py-styled-execute.exp @@ -13,7 +13,7 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . -# Check the the output of gdb.execute can be styled or not depending +# Check the output of gdb.execute can be styled or not depending # on the value of the third argument passed to gdb.execute. require allow_python_tests diff --git a/gdb/testsuite/lib/dwarf.exp b/gdb/testsuite/lib/dwarf.exp index a6a31916204..ab73bb4dff4 100644 --- a/gdb/testsuite/lib/dwarf.exp +++ b/gdb/testsuite/lib/dwarf.exp @@ -509,7 +509,7 @@ proc get_func_info { name {options {debug}} } { # * Otherwise, if the attribute name has a default form (f.i. DW_FORM_addr for # DW_AT_low_pc), then that one is used. # * Otherwise, an error is reported. Either specify a form explicitly, or -# add a default for the the attribute name in _default_form. +# add a default for the attribute name in _default_form. # # CHILDREN is just Tcl code that can be used to define child DIEs. It # is evaluated in the caller's context.