[0/9] Make gdb dir codespell-clean

Message ID	20260603100516.144737-1-tdevries@suse.de
Headers	DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 92EE94BA2E04 From: Tom de Vries <tdevries@suse.de> To: gdb-patches@sourceware.org Subject: [PATCH 0/9] [gdb] Make gdb dir codespell-clean Date: Wed, 3 Jun 2026 12:05:07 +0200 Message-ID: <20260603100516.144737-1-tdevries@suse.de> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit default: False [-3.01 / 50.00]; BAYES_HAM(-3.00)[100.00%]; MID_CONTAINS_FROM(1.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; R_MISSING_CHARSET(0.50)[]; R_DKIM_ALLOW(-0.20)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; MX_GOOD(-0.01)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; RBL_SPAMHAUS_BLOCKED_OPENRESOLVER(0.00)[2a07:de40:b281:104:10:150:64:97:from]; RCPT_COUNT_ONE(0.00)[1]; ARC_NA(0.00)[]; FUZZY_RATELIMITED(0.00)[rspamd.com]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; RCVD_TLS_ALL(0.00)[]; DKIM_TRACE(0.00)[suse.de:+]; RCVD_COUNT_TWO(0.00)[2]; FROM_EQ_ENVFROM(0.00)[]; FROM_HAS_DN(0.00)[]; MIME_TRACE(0.00)[0:+]; TO_DN_NONE(0.00)[]; PREVIOUSLY_DELIVERED(0.00)[gdb-patches@sourceware.org]; DNSWL_BLOCKED(0.00)[2a07:de40:b281:104:10:150:64:97:from]; RCVD_VIA_SMTP_AUTH(0.00)[]; DBL_BLOCKED_OPENRESOLVER(0.00)[suse.de:mid, suse.de:dkim, imap1.dmz-prg2.suse.org:helo, imap1.dmz-prg2.suse.org:rdns] Precedence: list Errors-To: gdb-patches-bounces~patchwork=sourceware.org@sourceware.org
Series	Make gdb dir codespell-clean \| [0/9] Make gdb dir codespell-clean [1/9] Drop executable mode in some files [2/9,gdb/contrib] Describe words in codespell-ignore-words.txt [3/9,gdb/contrib] Extend codespell-ignore-words.txt [4/9] Fix typos [5/9] Fix codespell false positives by ignoring [6/9] Fix codespell false positive by string splitting [7/9] Fix codespell false positives by renaming [8/9,pre-commit] Include gdb in codespell check [9/9,pre-commit] Simplify codespell configuration

Message ID

20260603100516.144737-1-tdevries@suse.de

Headers

DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 92EE94BA2E04
From: Tom de Vries <tdevries@suse.de>
To: gdb-patches@sourceware.org
Subject: [PATCH 0/9] [gdb] Make gdb dir codespell-clean
Date: Wed,  3 Jun 2026 12:05:07 +0200
Message-ID: <20260603100516.144737-1-tdevries@suse.de>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: list
Errors-To: gdb-patches-bounces~patchwork=sourceware.org@sourceware.org

Series

Make gdb dir codespell-clean |

Message

Tom de Vries June 3, 2026, 10:05 a.m. UTC

  This patch series make the gdb dir codespell-clean.

It starts out with two unrelated patches:
- the first fixes a problem found during review.
- the second adds brief description lines in codespell-ignore-words.txt.

A few following patches fix codespell errors, each in a different way.

The pre-last patch adds the gdb dir to the pre-commit codespell check.

The last patch move selection of files to check from .pre-commit-config.yaml
to gdb/pyproject.toml.

Tom de Vries (9):
  [gdb] Drop executable mode in some files
  [gdb/contrib] Describe words in codespell-ignore-words.txt
  [gdb/contrib] Extend codespell-ignore-words.txt
  [gdb] Fix typos
  [gdb] Fix codespell false positives by ignoring
  [gdb] Fix codespell false positive by string splitting
  [gdb] Fix codespell false positives by renaming
  [pre-commit] Include gdb in codespell check
  [pre-commit] Simplify codespell configuration

 .pre-commit-config.yaml                       |   2 +-
 gdb/ada-lang.c                                |   4 +-
 gdb/amd64-tdep.c                              |   2 +-
 gdb/arc-linux-tdep.c                          |   4 +
 gdb/arm-linux-nat.c                           |   6 +-
 gdb/arm-tdep.c                                |   2 +-
 gdb/avr-tdep.c                                |   5 +-
 gdb/bfin-linux-tdep.c                         |   2 +-
 gdb/bfin-tdep.c                               |   2 +
 gdb/buildsym.c                                |   6 +-
 gdb/buildsym.h                                |   2 +-
 gdb/completer.c                               |  10 +-
 gdb/completer.h                               |   2 +-
 gdb/contrib/codespell-ignore-words.txt        |  26 +++
 gdb/cp-support.c                              |   2 +-
 gdb/darwin-nat-info.c                         |   2 +-
 gdb/displaced-stepping.h                      |   2 +-
 gdb/gdbtypes.c                                |  16 +-
 gdb/i386-tdep.c                               | 170 +++++++++---------
 gdb/i387-tdep.c                               |   2 +
 gdb/infrun.c                                  |   2 +-
 gdb/infrun.h                                  |   2 +-
 gdb/loongarch-tdep.c                          |   2 +-
 gdb/mips-linux-nat.c                          |   2 +-
 gdb/mips-tdep.c                               |   2 +
 gdb/nds32-tdep.c                              |   2 +
 gdb/ppc-linux-nat.c                           |   4 +-
 gdb/ppc-linux-tdep.c                          |   8 +-
 gdb/printcmd.c                                |   6 +-
 gdb/pyproject.toml                            |   5 +-
 gdb/remote.c                                  |   4 +-
 gdb/riscv-tdep.c                              |   2 +
 gdb/s390-linux-tdep.c                         |   2 +
 gdb/s390-tdep.c                               |   6 +
 gdb/ser-event.c                               |  14 +-
 gdb/source.c                                  |   3 +-
 gdb/symtab.c                                  |   4 +-
 gdb/testsuite/gdb.arch/amd64-lam.c            |   0
 .../gdb.base/jump_multiple_objfiles-foo.c     |   0
 .../gdb.base/jump_multiple_objfiles.c         |   0
 .../gdb.base/jump_multiple_objfiles.exp       |   0
 .../gdb.base/jump_multiple_objfiles.h         |   0
 gdb/testsuite/gdb.btrace/exception.exp        |   0
 gdb/testsuite/gdb.fortran/nested-funcs.exp    |   0
 gdb/testsuite/gdb.fortran/nested-funcs.f90    |   0
 gdb/testsuite/gdb.fortran/oop_extend_type.exp |   0
 gdb/testsuite/gdb.fortran/oop_extend_type.f90 |   0
 gdb/testsuite/gdb.fortran/print_type.exp      |   0
 gdb/testsuite/gdb.fortran/vla-type.exp        |   0
 gdb/testsuite/gdb.fortran/vla-type.f90        |   0
 .../gdb.testsuite/gdb_test_multiple-lbl.gdb   |   0
 gdb/testsuite/lib/compiler.cc                 |   0
 gdb/utils.c                                   |   4 +-
 gdb/valprint.c                                |   8 +-
 gdb/value.c                                   |   3 +-
 gdb/windows-nat.c                             |  12 +-
 gdb/windows-nat.h                             |   2 +-
 gdb/x86-windows-nat.c                         |   2 +
 gdb/xcoffread.c                               |   2 +-
 gdb/xtensa-tdep.c                             |   2 +-
 gdb/xtensa-xtregs.c                           |   2 +
 gdb/z80-tdep.c                                |   2 +-
 62 files changed, 220 insertions(+), 158 deletions(-)
 mode change 100755 => 100644 gdb/amd64-tdep.c
 mode change 100755 => 100644 gdb/testsuite/gdb.arch/amd64-lam.c
 mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles-foo.c
 mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles.c
 mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles.exp
 mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles.h
 mode change 100755 => 100644 gdb/testsuite/gdb.btrace/exception.exp
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/nested-funcs.exp
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/nested-funcs.f90
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/oop_extend_type.exp
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/oop_extend_type.f90
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/print_type.exp
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/vla-type.exp
 mode change 100755 => 100644 gdb/testsuite/gdb.fortran/vla-type.f90
 mode change 100755 => 100644 gdb/testsuite/gdb.testsuite/gdb_test_multiple-lbl.gdb
 mode change 100755 => 100644 gdb/testsuite/lib/compiler.cc


base-commit: 22e21551bd7eb2fe14cdebfb4d89d51e5d93b7dd

Comments

Tom de Vries June 15, 2026, 10:29 a.m. UTC | #1

On 6/3/26 12:05 PM, Tom de Vries wrote:
> This patch series make the gdb dir codespell-clean.
> 
> It starts out with two unrelated patches:
> - the first fixes a problem found during review.
> - the second adds brief description lines in codespell-ignore-words.txt.
> 
> A few following patches fix codespell errors, each in a different way.
> 
> The pre-last patch adds the gdb dir to the pre-commit codespell check.
> 
> The last patch move selection of files to check from .pre-commit-config.yaml
> to gdb/pyproject.toml.
> 

With current trunk and this patch series, we get:
...
$ pre-commit run codespell --all-files
codespell................................................................Failed
- hook id: codespell
- exit code: 65

gdb/microblaze-linux-nat.c:97: Transfering ==> Transferring

$
...

Since the patch series is low-risk, I'm pushing this.

The only change I'm adding is this (and similar):
...
-The codespell:ignore-begin raison d'etre codespell:ignore-end of
+The <codespell:ignore-begin> raison d'etre <codespell:ignore-end> of
...
to make the sentence easier to read.

While re-reviewing this patch series, I've filed a PR to improve 
codespell support for gdb ( 
https://sourceware.org/bugzilla/show_bug.cgi?id=34285 ).

Thanks,
- Tom

> Tom de Vries (9):
>    [gdb] Drop executable mode in some files
>    [gdb/contrib] Describe words in codespell-ignore-words.txt
>    [gdb/contrib] Extend codespell-ignore-words.txt
>    [gdb] Fix typos
>    [gdb] Fix codespell false positives by ignoring
>    [gdb] Fix codespell false positive by string splitting
>    [gdb] Fix codespell false positives by renaming
>    [pre-commit] Include gdb in codespell check
>    [pre-commit] Simplify codespell configuration
> 
>   .pre-commit-config.yaml                       |   2 +-
>   gdb/ada-lang.c                                |   4 +-
>   gdb/amd64-tdep.c                              |   2 +-
>   gdb/arc-linux-tdep.c                          |   4 +
>   gdb/arm-linux-nat.c                           |   6 +-
>   gdb/arm-tdep.c                                |   2 +-
>   gdb/avr-tdep.c                                |   5 +-
>   gdb/bfin-linux-tdep.c                         |   2 +-
>   gdb/bfin-tdep.c                               |   2 +
>   gdb/buildsym.c                                |   6 +-
>   gdb/buildsym.h                                |   2 +-
>   gdb/completer.c                               |  10 +-
>   gdb/completer.h                               |   2 +-
>   gdb/contrib/codespell-ignore-words.txt        |  26 +++
>   gdb/cp-support.c                              |   2 +-
>   gdb/darwin-nat-info.c                         |   2 +-
>   gdb/displaced-stepping.h                      |   2 +-
>   gdb/gdbtypes.c                                |  16 +-
>   gdb/i386-tdep.c                               | 170 +++++++++---------
>   gdb/i387-tdep.c                               |   2 +
>   gdb/infrun.c                                  |   2 +-
>   gdb/infrun.h                                  |   2 +-
>   gdb/loongarch-tdep.c                          |   2 +-
>   gdb/mips-linux-nat.c                          |   2 +-
>   gdb/mips-tdep.c                               |   2 +
>   gdb/nds32-tdep.c                              |   2 +
>   gdb/ppc-linux-nat.c                           |   4 +-
>   gdb/ppc-linux-tdep.c                          |   8 +-
>   gdb/printcmd.c                                |   6 +-
>   gdb/pyproject.toml                            |   5 +-
>   gdb/remote.c                                  |   4 +-
>   gdb/riscv-tdep.c                              |   2 +
>   gdb/s390-linux-tdep.c                         |   2 +
>   gdb/s390-tdep.c                               |   6 +
>   gdb/ser-event.c                               |  14 +-
>   gdb/source.c                                  |   3 +-
>   gdb/symtab.c                                  |   4 +-
>   gdb/testsuite/gdb.arch/amd64-lam.c            |   0
>   .../gdb.base/jump_multiple_objfiles-foo.c     |   0
>   .../gdb.base/jump_multiple_objfiles.c         |   0
>   .../gdb.base/jump_multiple_objfiles.exp       |   0
>   .../gdb.base/jump_multiple_objfiles.h         |   0
>   gdb/testsuite/gdb.btrace/exception.exp        |   0
>   gdb/testsuite/gdb.fortran/nested-funcs.exp    |   0
>   gdb/testsuite/gdb.fortran/nested-funcs.f90    |   0
>   gdb/testsuite/gdb.fortran/oop_extend_type.exp |   0
>   gdb/testsuite/gdb.fortran/oop_extend_type.f90 |   0
>   gdb/testsuite/gdb.fortran/print_type.exp      |   0
>   gdb/testsuite/gdb.fortran/vla-type.exp        |   0
>   gdb/testsuite/gdb.fortran/vla-type.f90        |   0
>   .../gdb.testsuite/gdb_test_multiple-lbl.gdb   |   0
>   gdb/testsuite/lib/compiler.cc                 |   0
>   gdb/utils.c                                   |   4 +-
>   gdb/valprint.c                                |   8 +-
>   gdb/value.c                                   |   3 +-
>   gdb/windows-nat.c                             |  12 +-
>   gdb/windows-nat.h                             |   2 +-
>   gdb/x86-windows-nat.c                         |   2 +
>   gdb/xcoffread.c                               |   2 +-
>   gdb/xtensa-tdep.c                             |   2 +-
>   gdb/xtensa-xtregs.c                           |   2 +
>   gdb/z80-tdep.c                                |   2 +-
>   62 files changed, 220 insertions(+), 158 deletions(-)
>   mode change 100755 => 100644 gdb/amd64-tdep.c
>   mode change 100755 => 100644 gdb/testsuite/gdb.arch/amd64-lam.c
>   mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles-foo.c
>   mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles.c
>   mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles.exp
>   mode change 100755 => 100644 gdb/testsuite/gdb.base/jump_multiple_objfiles.h
>   mode change 100755 => 100644 gdb/testsuite/gdb.btrace/exception.exp
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/nested-funcs.exp
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/nested-funcs.f90
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/oop_extend_type.exp
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/oop_extend_type.f90
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/print_type.exp
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/vla-type.exp
>   mode change 100755 => 100644 gdb/testsuite/gdb.fortran/vla-type.f90
>   mode change 100755 => 100644 gdb/testsuite/gdb.testsuite/gdb_test_multiple-lbl.gdb
>   mode change 100755 => 100644 gdb/testsuite/lib/compiler.cc
> 
> 
> base-commit: 22e21551bd7eb2fe14cdebfb4d89d51e5d93b7dd