Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [pushed 1/3] [gdb] Fix some typos
@ 2026-09-03  5:47 Tom de Vries
  2026-09-03  5:47 ` [pushed 2/3] [gdb] Fix double words Tom de Vries
  2026-09-03  5:47 ` [pushed 3/3] [gdb] Fix dont -> don't typo Tom de Vries
  0 siblings, 2 replies; 3+ messages in thread
From: Tom de Vries @ 2026-09-03  5:47 UTC (permalink / raw)
  To: gdb-patches

Fix three typos I found running:
...
$ ./gdb/contrib/words.sh \
      -c \
      $(find gdb -type f -name "*.c" -o -name "*.h" -o -name "*.cc") \
      | codespell -
...
---
 gdb/procfs.c      | 2 +-
 gdb/rs6000-tdep.c | 2 +-
 gdb/tracepoint.c  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/gdb/procfs.c b/gdb/procfs.c
index bbee28bbc5e..615b92006a8 100644
--- a/gdb/procfs.c
+++ b/gdb/procfs.c
@@ -891,7 +891,7 @@ proc_wait_for_stop (procinfo *pi)
 
   clear_sigint_trap ();
 
-  /* We been runnin' and we stopped -- need to update status.  */
+  /* We've been running and we stopped -- need to update status.  */
   pi->status_valid = 0;
 
   return win;
diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index e7d1ec07c97..05a4f1a371c 100644
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -697,7 +697,7 @@ insn_changes_sp_or_jumps (unsigned long insn)
 
   /* NOTE: There are many ways to change the value of a given register.
 	   The ways below are those used when the register is R1, the SP,
-	   in a funtion's epilogue.  */
+	   in a function's epilogue.  */
 
   if (opcode == 31 && subcode == 444 && a == 1)
     return 1;  /* mr R1,Rn */
diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c
index 932a8f557e4..bedeb293366 100644
--- a/gdb/tracepoint.c
+++ b/gdb/tracepoint.c
@@ -873,7 +873,7 @@ collection_list::add_local_register (struct gdbarch *gdbarch,
 
       add_ax_registers (aexpr.get ());
 
-      /* Usually ax_reg_mask for a pseudo-regiser only sets the
+      /* Usually ax_reg_mask for a pseudo-register only sets the
 	 corresponding raw registers in the ax mask, but if this isn't
 	 the case add the expression that is generated to the
 	 collection list.  */

base-commit: 1c948f2e10ecde66b0ee0bb77519e272a660e906
-- 
2.51.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [pushed 2/3] [gdb] Fix double words
  2026-09-03  5:47 [pushed 1/3] [gdb] Fix some typos Tom de Vries
@ 2026-09-03  5:47 ` Tom de Vries
  2026-09-03  5:47 ` [pushed 3/3] [gdb] Fix dont -> don't typo Tom de Vries
  1 sibling, 0 replies; 3+ messages in thread
From: Tom de Vries @ 2026-09-03  5:47 UTC (permalink / raw)
  To: gdb-patches

Result of:
...
$ find gdb* -type f | xargs sed -i "s/ is is / is /"
...
and likewise for with, of, we, be, by, a.

Handle "to to" manually, one of those is fixed as "to do".
---
 gdb/NEWS                                            | 2 +-
 gdb/README                                          | 2 +-
 gdb/arm-tdep.c                                      | 2 +-
 gdb/avr-tdep.c                                      | 2 +-
 gdb/breakpoint.c                                    | 4 ++--
 gdb/breakpoint.h                                    | 2 +-
 gdb/c-typeprint.c                                   | 2 +-
 gdb/doc/python.texi                                 | 4 ++--
 gdb/dwarf2/unit-head.c                              | 2 +-
 gdb/frame.h                                         | 2 +-
 gdb/frv-linux-tdep.c                                | 2 +-
 gdb/gdbthread.h                                     | 2 +-
 gdb/infrun.c                                        | 2 +-
 gdb/m68k-linux-tdep.c                               | 2 +-
 gdb/or1k-tdep.c                                     | 2 +-
 gdb/python/lib/gdb/dap/varref.py                    | 2 +-
 gdb/python/python-internal.h                        | 2 +-
 gdb/remote.c                                        | 4 ++--
 gdb/remote.h                                        | 2 +-
 gdb/svr4-tls-tdep.c                                 | 2 +-
 gdb/testsuite/gdb.base/call-sc.exp                  | 2 +-
 gdb/testsuite/gdb.base/watchpoint.exp               | 2 +-
 gdb/testsuite/gdb.dwarf2/backward-spec-inter-cu.exp | 2 +-
 gdb/testsuite/gdb.dwarf2/forward-spec-inter-cu.exp  | 2 +-
 gdb/testsuite/gdb.dwarf2/forward-spec.exp           | 2 +-
 gdb/varobj.c                                        | 2 +-
 gdbserver/server.cc                                 | 4 ++--
 27 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/gdb/NEWS b/gdb/NEWS
index 10c182067f9..222f38e3d52 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -5812,7 +5812,7 @@ Renesas RL78				rl78-*-elf
      This keyword, when provided, will direct the output to either
      stdout, stderr, or GDB's logging output.
 
-  ** Parameters can now be be sub-classed in Python, and in particular
+  ** Parameters can now be sub-classed in Python, and in particular
      you may implement the get_set_doc and get_show_doc functions.
      This improves how Parameter set/show documentation is processed
      and allows for more dynamic content.
diff --git a/gdb/README b/gdb/README
index 791bbd23b6c..81e3cd712a7 100644
--- a/gdb/README
+++ b/gdb/README
@@ -418,7 +418,7 @@ more obscure GDB `configure' options are not listed here.
 
 `--enable-binary-file-formats=FORMAT,FORMAT,...'
 `--enable-binary-file-formats=all'
-    Configure GDB to only be be able to read selected file formats.
+    Configure GDB to only be able to read selected file formats.
     The special value "all" causes all file formats to be compiled
     in, and is the default behavior of the option.  This option
     is meant for advanced users who are sure of what they expect,
diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index c51074c89ff..7ac544c3bbf 100644
--- a/gdb/arm-tdep.c
+++ b/gdb/arm-tdep.c
@@ -8751,7 +8751,7 @@ gdb_print_insn_arm (bfd_vma memaddr, disassemble_info *info)
 
    There are other ways of forcing a breakpoint.  GNU/Linux, RISC iX,
    and NetBSD all use a software interrupt rather than an undefined
-   instruction to force a trap.  This can be handled by by the
+   instruction to force a trap.  This can be handled by the
    abi-specific code during establishment of the gdbarch vector.  */
 
 #define ARM_LE_BREAKPOINT {0xFE,0xDE,0xFF,0xE7}
diff --git a/gdb/avr-tdep.c b/gdb/avr-tdep.c
index 012022f2b73..95726d3eb0f 100644
--- a/gdb/avr-tdep.c
+++ b/gdb/avr-tdep.c
@@ -535,7 +535,7 @@ avr_scan_prologue (struct gdbarch *gdbarch, CORE_ADDR pc_beg, CORE_ADDR pc_end,
 
   /* FIXME: TRoth/2003-06-11: This could be made more efficient by only
      reading in the bytes of the prologue.  The problem is that the figuring
-     out where the end of the prologue is is a bit difficult.  The old code
+     out where the end of the prologue is a bit difficult.  The old code
      tried to do that, but failed quite often.  */
   read_memory (pc_beg, prologue, len);
 
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index ca65d7e135d..b2a432e26ba 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -7373,7 +7373,7 @@ watchpoint_locations_match (const struct bp_location *loc1,
   gdb_assert (w2 != NULL);
 
   /* If the target can evaluate the condition expression in hardware,
-     then we we need to insert both watchpoints even if they are at
+     then we need to insert both watchpoints even if they are at
      the same place.  Otherwise the watchpoint will only trigger when
      the condition of whichever watchpoint was inserted evaluates to
      true, not giving a chance for GDB to check the condition of the
@@ -9139,7 +9139,7 @@ decode_static_tracepoint_spec (const char **arg_p)
   return sals;
 }
 
-/* Returns the breakpoint ops appropriate for use with with LOCATION_TYPE and
+/* Returns the breakpoint ops appropriate for use with LOCATION_TYPE and
    according to IS_TRACEPOINT.  */
 
 static const struct breakpoint_ops *
diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
index 722d75390fa..75de448b16e 100644
--- a/gdb/breakpoint.h
+++ b/gdb/breakpoint.h
@@ -1604,7 +1604,7 @@ extern void
 extern breakpoint *install_breakpoint
   (int internal, std::unique_ptr<breakpoint> &&b, int update_gll);
 
-/* Returns the breakpoint ops appropriate for use with with LOCSPEC
+/* Returns the breakpoint ops appropriate for use with LOCSPEC
    and according to IS_TRACEPOINT.  Use this to ensure, for example,
    that you pass the correct ops to create_breakpoint for probe
    location specs.  If LOCSPEC is NULL, returns
diff --git a/gdb/c-typeprint.c b/gdb/c-typeprint.c
index 07f01c92b15..00805e029f8 100644
--- a/gdb/c-typeprint.c
+++ b/gdb/c-typeprint.c
@@ -336,7 +336,7 @@ cp_type_print_method_args (struct type *mtype,
    any typename for TYPE and show its details.
    SHOW is always zero on recursive calls.
 
-   NEED_POST_SPACE is non-zero when a space will be be needed
+   NEED_POST_SPACE is non-zero when a space will be needed
    between a trailing qualifier and a field, variable, or function
    name.  */
 
diff --git a/gdb/doc/python.texi b/gdb/doc/python.texi
index 6cafab78d58..1a38360ca1a 100644
--- a/gdb/doc/python.texi
+++ b/gdb/doc/python.texi
@@ -394,7 +394,7 @@ Takes @var{value}, an instance of @code{gdb.Value} (@pxref{Values From
 Inferior}), and appends the value this object represents to
 @value{GDBN}'s value history (@pxref{Value History}), and return an
 integer, its history number.  If @var{value} is not a
-@code{gdb.Value}, it is is converted using the @code{gdb.Value}
+@code{gdb.Value}, it is converted using the @code{gdb.Value}
 constructor.  If @var{value} can't be converted to a @code{gdb.Value}
 then a @code{TypeError} is raised.
 
@@ -422,7 +422,7 @@ named @var{name}.  @var{name} must be a string.  The name should not
 include the @samp{$} that is used to mark a convenience variable in an
 expression.  If @var{value} is @code{None}, then the convenience
 variable is removed.  Otherwise, if @var{value} is not a
-@code{gdb.Value} (@pxref{Values From Inferior}), it is is converted
+@code{gdb.Value} (@pxref{Values From Inferior}), it is converted
 using the @code{gdb.Value} constructor.
 @end defun
 
diff --git a/gdb/dwarf2/unit-head.c b/gdb/dwarf2/unit-head.c
index 9d8818e6917..1771e43da97 100644
--- a/gdb/dwarf2/unit-head.c
+++ b/gdb/dwarf2/unit-head.c
@@ -142,7 +142,7 @@ read_unit_head (struct unit_head *header, const gdb_byte *info_ptr,
   return info_ptr;
 }
 
-/* Subroutine of read_and_check_unit_head to to simplify it.
+/* Subroutine of read_and_check_unit_head to simplify it.
    Perform various error checking on the header.  */
 
 static void
diff --git a/gdb/frame.h b/gdb/frame.h
index 33a08b1403b..b64650f3215 100644
--- a/gdb/frame.h
+++ b/gdb/frame.h
@@ -259,7 +259,7 @@ class frame_info_ptr : public intrusive_list_node<frame_info_ptr>
 
   ~frame_info_ptr ()
   {
-    /* If this node has static storage, it should be be deleted before
+    /* If this node has static storage, it should be deleted before
        frame_list.  */
     frame_list.erase (frame_list.iterator_to (*this));
   }
diff --git a/gdb/frv-linux-tdep.c b/gdb/frv-linux-tdep.c
index 956990824cf..c9a55f0ad43 100644
--- a/gdb/frv-linux-tdep.c
+++ b/gdb/frv-linux-tdep.c
@@ -275,7 +275,7 @@ frv_linux_sigtramp_frame_cache (const frame_info_ptr &this_frame,
 
   cache = trad_frame_cache_zalloc (this_frame);
 
-  /* FIXME: cagney/2004-05-01: This is is long standing broken code.
+  /* FIXME: cagney/2004-05-01: This is long standing broken code.
      The frame ID's code address should be the start-address of the
      signal trampoline and not the current PC within that
      trampoline.  */
diff --git a/gdb/gdbthread.h b/gdb/gdbthread.h
index 5e540f9d09a..51e9ae1b3f0 100644
--- a/gdb/gdbthread.h
+++ b/gdb/gdbthread.h
@@ -355,7 +355,7 @@ class thread_info : public intrusive_list_node<thread_info>,
 	  per-inferior thread number.  This number is unique in the
 	  inferior but not unique between inferiors.
 
-     b) The global ID (GId).  This is a a single integer unique
+     b) The global ID (GId).  This is a single integer unique
 	between all inferiors.
 
      E.g.:
diff --git a/gdb/infrun.c b/gdb/infrun.c
index c0767e7f764..bc88e40b8d5 100644
--- a/gdb/infrun.c
+++ b/gdb/infrun.c
@@ -1718,7 +1718,7 @@ infrun_inferior_execd (inferior *exec_inf, inferior *follow_inf)
 {
   /* If some threads where was doing a displaced step in this inferior at the
      moment of the exec, they no longer exist.  Even if the exec'ing thread
-     doing a displaced step, we don't want to to any fixup nor restore displaced
+     doing a displaced step, we don't want to do any fixup nor restore displaced
      stepping buffer bytes.  */
   follow_inf->displaced_step_state.reset ();
 
diff --git a/gdb/m68k-linux-tdep.c b/gdb/m68k-linux-tdep.c
index ceb15cb1a20..81e957dc681 100644
--- a/gdb/m68k-linux-tdep.c
+++ b/gdb/m68k-linux-tdep.c
@@ -265,7 +265,7 @@ m68k_linux_sigtramp_frame_cache (const frame_info_ptr &this_frame,
 
   cache = trad_frame_cache_zalloc (this_frame);
 
-  /* FIXME: cagney/2004-05-01: This is is long standing broken code.
+  /* FIXME: cagney/2004-05-01: This is long standing broken code.
      The frame ID's code address should be the start-address of the
      signal trampoline and not the current PC within that
      trampoline.  */
diff --git a/gdb/or1k-tdep.c b/gdb/or1k-tdep.c
index 1351dbe2d0c..1935bedc82c 100644
--- a/gdb/or1k-tdep.c
+++ b/gdb/or1k-tdep.c
@@ -198,7 +198,7 @@ or1k_analyse_l_addi (uint32_t inst, unsigned int *rd_ptr,
     return false; /* Failure */
 }
 
-/* This is used to to parse store instructions during various prologue
+/* This is used to parse store instructions during various prologue
    analysis routines.  The l.sw instruction has semantics:
 
      assembly:        l.sw  I(rA),rB
diff --git a/gdb/python/lib/gdb/dap/varref.py b/gdb/python/lib/gdb/dap/varref.py
index 3cd1ce74fbf..e2dcd03910c 100644
--- a/gdb/python/lib/gdb/dap/varref.py
+++ b/gdb/python/lib/gdb/dap/varref.py
@@ -119,7 +119,7 @@ class BaseReference(ABC):
     # Helper method to compute the final name for a child whose base
     # name is given.  Updates the name_counts map.  This is used to
     # handle shadowing -- in DAP, the adapter is responsible for
-    # making sure that all the variables in a a given container have
+    # making sure that all the variables in a given container have
     # unique names.  See
     # https://github.com/microsoft/debug-adapter-protocol/issues/141
     # and
diff --git a/gdb/python/python-internal.h b/gdb/python/python-internal.h
index 5529e9fd4d7..57288b9ead3 100644
--- a/gdb/python/python-internal.h
+++ b/gdb/python/python-internal.h
@@ -1201,7 +1201,7 @@ class gdbpy_registry
   }
 
   /* Unregister Python object OBJ.  OBJ will no longer be invalidated when
-     OWNER is about to be be freed.  */
+     OWNER is about to be freed.  */
   template <typename O>
   void remove (O *owner, obj_type *obj) const
   {
diff --git a/gdb/remote.c b/gdb/remote.c
index 30d0d171d1f..76cf6b5234f 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -6748,7 +6748,7 @@ thread_pending_fork_status (struct thread_info *thread)
   return &ws;
 }
 
-/* Return THREAD's pending status if is is a pending fork/vfork/clone
+/* Return THREAD's pending status if is a pending fork/vfork/clone
    event, else return nullptr.  */
 
 static const target_waitstatus *
@@ -10822,7 +10822,7 @@ show_watchdog (struct ui_file *file, int from_tty,
 /* Read a packet from the remote machine, with error checking, and
    store it in *BUF.  Resize *BUF if necessary to hold the result.  If
    FOREVER, wait forever rather than timing out; this is used (in
-   synchronous mode) to wait for a target that is is executing user
+   synchronous mode) to wait for a target that is executing user
    code to stop.  If FOREVER == false, this function is allowed to time
    out gracefully and return an indication of this to the caller.
    Otherwise return the number of bytes read.  If IS_NOTIF is not
diff --git a/gdb/remote.h b/gdb/remote.h
index 5597dd74af8..4e61c9c88e3 100644
--- a/gdb/remote.h
+++ b/gdb/remote.h
@@ -50,7 +50,7 @@ extern bool remote_debug;
    store it in *BUF.  Resize *BUF using xrealloc if necessary to hold
    the result, and update *SIZEOF_BUF.  If FOREVER, wait forever
    rather than timing out; this is used (in synchronous mode) to wait
-   for a target that is is executing user code to stop.  */
+   for a target that is executing user code to stop.  */
 
 extern void getpkt (remote_target *remote,
 		    char **buf, long *sizeof_buf, int forever);
diff --git a/gdb/svr4-tls-tdep.c b/gdb/svr4-tls-tdep.c
index b39623b324c..0fd09f46e93 100644
--- a/gdb/svr4-tls-tdep.c
+++ b/gdb/svr4-tls-tdep.c
@@ -117,7 +117,7 @@ svr4_tls_get_thread_local_address (struct gdbarch *gdbarch, ptid_t ptid,
        target stack, except when 'force_internal_tls_address_lookup'
        has been set.
 
-     The idea here is to prefer use of of the target's thread_stratum
+     The idea here is to prefer use of the target's thread_stratum
      method since it should be more accurate.  */
   if (gdbarch_data->get_tls_dtv_addr == nullptr
       || (current_inferior ()->target_at (thread_stratum) != nullptr
diff --git a/gdb/testsuite/gdb.base/call-sc.exp b/gdb/testsuite/gdb.base/call-sc.exp
index 9f9019058ef..825c91c1937 100644
--- a/gdb/testsuite/gdb.base/call-sc.exp
+++ b/gdb/testsuite/gdb.base/call-sc.exp
@@ -230,7 +230,7 @@ proc test_scalar_returns { } {
 
     # If the previous test did not work, the program counter might
     # still be inside foo() rather than main().  Make sure the program
-    # counter is is main().
+    # counter is main().
     #
     # This happens on ppc64 GNU/Linux with gcc 3.4.1 and a buggy GDB
 
diff --git a/gdb/testsuite/gdb.base/watchpoint.exp b/gdb/testsuite/gdb.base/watchpoint.exp
index 741e7e441bf..cf6edce23c0 100644
--- a/gdb/testsuite/gdb.base/watchpoint.exp
+++ b/gdb/testsuite/gdb.base/watchpoint.exp
@@ -438,7 +438,7 @@ proc test_watchpoint_triggered_in_syscall {} {
     }
 }
 
-# Do a simple test of of watching through a pointer when the pointer
+# Do a simple test of watching through a pointer when the pointer
 # itself changes.  Should add some more complicated stuff here.
 
 proc test_complex_watchpoint {} {
diff --git a/gdb/testsuite/gdb.dwarf2/backward-spec-inter-cu.exp b/gdb/testsuite/gdb.dwarf2/backward-spec-inter-cu.exp
index de76cc8aca2..2b59c01474b 100644
--- a/gdb/testsuite/gdb.dwarf2/backward-spec-inter-cu.exp
+++ b/gdb/testsuite/gdb.dwarf2/backward-spec-inter-cu.exp
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# Check that the DWARF reader works with a a DW_AT_specification that
+# Check that the DWARF reader works with a DW_AT_specification that
 # refers to an earlier DIE.  Inter-cu variant of forward-spec.exp.
 
 load_lib dwarf.exp
diff --git a/gdb/testsuite/gdb.dwarf2/forward-spec-inter-cu.exp b/gdb/testsuite/gdb.dwarf2/forward-spec-inter-cu.exp
index eb1d8a2df58..a2c5f7b921e 100644
--- a/gdb/testsuite/gdb.dwarf2/forward-spec-inter-cu.exp
+++ b/gdb/testsuite/gdb.dwarf2/forward-spec-inter-cu.exp
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# Check that the DWARF reader works with a a DW_AT_specification that
+# Check that the DWARF reader works with a DW_AT_specification that
 # refers to a later DIE.  Inter-cu variant of forward-spec.exp.
 
 load_lib dwarf.exp
diff --git a/gdb/testsuite/gdb.dwarf2/forward-spec.exp b/gdb/testsuite/gdb.dwarf2/forward-spec.exp
index fd7d008adb3..80156dbd9a1 100644
--- a/gdb/testsuite/gdb.dwarf2/forward-spec.exp
+++ b/gdb/testsuite/gdb.dwarf2/forward-spec.exp
@@ -13,7 +13,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# Check that the DWARF reader works with a a DW_AT_specification that
+# Check that the DWARF reader works with a DW_AT_specification that
 # refers to a later DIE.
 
 require !readnow
diff --git a/gdb/varobj.c b/gdb/varobj.c
index b1429bfe69a..edc0d48bf3b 100644
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -2263,7 +2263,7 @@ varobj_value_is_changeable_p (const struct varobj *var)
   return var->root->lang_ops->value_is_changeable_p (var);
 }
 
-/* Return true if that varobj is floating, that is is always evaluated in the
+/* Return true if that varobj is floating, that is always evaluated in the
    selected frame, and not bound to thread/frame.  Such variable objects
    are created using '@' as frame specifier to -var-create.  */
 bool
diff --git a/gdbserver/server.cc b/gdbserver/server.cc
index 4a727c465f0..a208e24d09f 100644
--- a/gdbserver/server.cc
+++ b/gdbserver/server.cc
@@ -1318,7 +1318,7 @@ handle_detach (char *own_buf)
 		 "Persistent commands are present, "
 		 "leaving gdbserver attached to the process\n");
 
-      /* Make sure we're in non-stop/async mode, so we we can both
+      /* Make sure we're in non-stop/async mode, so we can both
 	 wait for an async socket accept, and handle async target
 	 events simultaneously.  There's also no point either in
 	 having the target stop all threads, when we're going to
@@ -4676,7 +4676,7 @@ captured_main (int argc, char *argv[])
 	    {
 	      if (disconnected_tracing)
 		{
-		  /* Try to enable non-stop/async mode, so we we can
+		  /* Try to enable non-stop/async mode, so we can
 		     both wait for an async socket accept, and handle
 		     async target events simultaneously.  There's also
 		     no point either in having the target always stop
-- 
2.51.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [pushed 3/3] [gdb] Fix dont -> don't typo
  2026-09-03  5:47 [pushed 1/3] [gdb] Fix some typos Tom de Vries
  2026-09-03  5:47 ` [pushed 2/3] [gdb] Fix double words Tom de Vries
@ 2026-09-03  5:47 ` Tom de Vries
  1 sibling, 0 replies; 3+ messages in thread
From: Tom de Vries @ 2026-09-03  5:47 UTC (permalink / raw)
  To: gdb-patches

Result of:
...
$ find gdb* -type f | xargs sed -i "s/ dont / don't /"
...
and manually fixing two hits of ' dont.'

The "dont->don't" rule is part of the code dictionary:
...
- 'code' for words from code and/or mathematics that are likely to be typos
  in other contexts (such as uint)
...
but it tries to rewrite stdio into studio, so we'll leave that disabled.

We do have a fair amount of dont_<something> in our sources, so I'm not adding
this as a rule in gdb/contrib/codespell-dictionary.txt.  It doesn't hurt us
now, but it does if we start playing with the word regexp [1].

[1] https://sourceware.org/bugzilla/show_bug.cgi?id=34241
---
 gdb/arm-tdep.c                             | 2 +-
 gdb/dwarf2/loc.c                           | 4 ++--
 gdb/m32r-tdep.c                            | 2 +-
 gdb/mips-tdep.c                            | 2 +-
 gdb/stubs/m68k-stub.c                      | 2 +-
 gdb/testsuite/gdb.base/skip-inline.exp     | 2 +-
 gdb/testsuite/gdb.reverse/step-reverse.exp | 2 +-
 gdbsupport/event-loop.cc                   | 2 +-
 8 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index 7ac544c3bbf..bffd5d22199 100644
--- a/gdb/arm-tdep.c
+++ b/gdb/arm-tdep.c
@@ -11546,7 +11546,7 @@ arm_record_extension_space (arm_insn_decode_record *arm_insn_r)
 	      if (0 == insn_op1 || 1 == insn_op1)
 		{
 		  /* SMLA<x><y>, SMLAW<y>, SMULW<y>.  */
-		  /* We dont do optimization for SMULW<y> where we
+		  /* We don't do optimization for SMULW<y> where we
 		     need only Rd.  */
 		  record_buf[0] = bits (arm_insn_r->arm_insn, 12, 15);
 		  record_buf[1] = ARM_PS_REGNUM;
diff --git a/gdb/dwarf2/loc.c b/gdb/dwarf2/loc.c
index 8e2b401ba34..15336aaf017 100644
--- a/gdb/dwarf2/loc.c
+++ b/gdb/dwarf2/loc.c
@@ -2161,7 +2161,7 @@ dwarf2_get_symbol_read_needs (gdb::array_view<const gdb_byte> expr,
 					     get_frame_pc);
 
 	    /* If SYMBOL_NEEDS_FRAME is returned from the previous call,
-	       we dont have to check the baton content.  */
+	       we don't have to check the baton content.  */
 	    if (symbol_needs != SYMBOL_NEEDS_FRAME)
 	      {
 		gdbarch *arch = baton.per_objfile->objfile->arch ();
@@ -2212,7 +2212,7 @@ dwarf2_get_symbol_read_needs (gdb::array_view<const gdb_byte> expr,
 					       get_frame_pc, true);
 
 	    /* If SYMBOL_NEEDS_FRAME is returned from the previous call,
-	       we dont have to check the baton content.  */
+	       we don't have to check the baton content.  */
 	    if (symbol_needs != SYMBOL_NEEDS_FRAME)
 	      {
 		gdbarch *arch = baton.per_objfile->objfile->arch ();
diff --git a/gdb/m32r-tdep.c b/gdb/m32r-tdep.c
index b5792eeeafc..d881f33c057 100644
--- a/gdb/m32r-tdep.c
+++ b/gdb/m32r-tdep.c
@@ -289,7 +289,7 @@ decode_prologue (struct gdbarch *gdbarch,
       if (insn == 0x0000)
 	break;
 
-      /* If this is a 32 bit instruction, we dont want to examine its
+      /* If this is a 32 bit instruction, we don't want to examine its
 	 immediate data as though it were an instruction.  */
       if (current_pc & 0x02)
 	{
diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index fe0482fe5bf..2a60d6ef097 100644
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -2120,7 +2120,7 @@ micromips_next_pc (struct regcache *regcache, CORE_ADDR pc)
 /* Decoding the next place to set a breakpoint is irregular for the
    mips 16 variant, but fortunately, there fewer instructions.  We have
    to cope ith extensions for 16 bit instructions and a pair of actual
-   32 bit instructions.  We dont want to set a single step instruction
+   32 bit instructions.  We don't want to set a single step instruction
    on the extend instruction either.  */
 
 /* Lots of mips16 instruction formats */
diff --git a/gdb/stubs/m68k-stub.c b/gdb/stubs/m68k-stub.c
index cbea4772fc7..a5ec9bce088 100644
--- a/gdb/stubs/m68k-stub.c
+++ b/gdb/stubs/m68k-stub.c
@@ -486,7 +486,7 @@ _returnFromException (Frame * frame)
       frame = lastFrame;
       frame->frameSize = 4;
       frame->format = 0;
-      frame->fsaveHeader = -1;	/* restore regs, but we dont have fsave info */
+      frame->fsaveHeader = -1;	/* restore regs, but we don't have fsave info */
     }
 
 #if !defined (mc68020) && !defined (mc68332)
diff --git a/gdb/testsuite/gdb.base/skip-inline.exp b/gdb/testsuite/gdb.base/skip-inline.exp
index a0223efa2eb..d15ba556044 100644
--- a/gdb/testsuite/gdb.base/skip-inline.exp
+++ b/gdb/testsuite/gdb.base/skip-inline.exp
@@ -94,7 +94,7 @@ gdb_test "skip function foo" "Function foo will be skipped when stepping\."
 single_step
 
 # Some compilers link the epilogue of functions to the line containing the
-# closing brace, while others dont.  The behavior ends up so out of sync
+# closing brace, while others don't.  The behavior ends up so out of sync
 # with multiple steps at a time, that completely different procs would need
 # to be made, and it is frankly unnecessary.
 if {$epilogue} {
diff --git a/gdb/testsuite/gdb.reverse/step-reverse.exp b/gdb/testsuite/gdb.reverse/step-reverse.exp
index d72638c9169..66ae378a153 100644
--- a/gdb/testsuite/gdb.reverse/step-reverse.exp
+++ b/gdb/testsuite/gdb.reverse/step-reverse.exp
@@ -174,7 +174,7 @@ gdb_test_multiple "stepi" "$test_message" {
 
 # stepi backward out of a function call
 
-# When testing stepi, we dont want to infinitely step if we're not moving
+# When testing stepi, we don't want to infinitely step if we're not moving
 # so we store the starting PC, in case we land on the same line as above
 set start_pc [get_hexadecimal_valueof "\$pc" 0 "get PC at start"]
 set get_pc_count 0
diff --git a/gdbsupport/event-loop.cc b/gdbsupport/event-loop.cc
index c6e1b5b4634..b7217282110 100644
--- a/gdbsupport/event-loop.cc
+++ b/gdbsupport/event-loop.cc
@@ -623,7 +623,7 @@ gdb_wait_for_event (int block)
 	  FD_ZERO (&gdb_notifier.ready_masks[1]);
 	  FD_ZERO (&gdb_notifier.ready_masks[2]);
 
-	  /* Dont print anything if we got a signal, let gdb handle
+	  /* Don't print anything if we got a signal, let gdb handle
 	     it.  */
 	  if (errno != EINTR)
 	    perror_with_name (("select"));
-- 
2.51.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-09-03  5:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-03  5:47 [pushed 1/3] [gdb] Fix some typos Tom de Vries
2026-09-03  5:47 ` [pushed 2/3] [gdb] Fix double words Tom de Vries
2026-09-03  5:47 ` [pushed 3/3] [gdb] Fix dont -> don't typo Tom de Vries

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox