]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
gdb: allow quoted filenames for commands that have custom completion
authorAndrew Burgess <aburgess@redhat.com>
Thu, 20 Jun 2024 12:44:28 +0000 (13:44 +0100)
committerAndrew Burgess <aburgess@redhat.com>
Sat, 7 Sep 2024 19:28:59 +0000 (20:28 +0100)
commite454ae416a050496107f08cb0ae0de518d732a90
tree13622c82f13832412cf8de08c02ff55f28960b1f
parentd552429edae9a9cb6d5c6506149fff1aa407daeb
gdb: allow quoted filenames for commands that have custom completion

This commit changes how GDB processes command arguments for the
following commands:

  compile file
  maint print c-tdesc
  save gdb-index

After this commit these commands will now expect their single filename
argument to be (optionally) quoted if it contains any special
characters (e.g. whit space or quotes).

If the filename does not contain any special characters then nothing
changes.  As an example:

   (gdb) save gdb-index /path/to/some/directory/

will work before and after this patch.  However, if the directory
name contains a white space then before this patch a user would write:

  (gdb) save gdb-index /path/to some/directory/

But this will now fail as GDB will consider this as two arguments,
'/path/to' and 'some/directory/'.  To pass this single directory name
a user must now do one of these:

  (gdb) save gdb-index "/path/to some/directory/"
  (gdb) save gdb-index '/path/to some/directory/'
  (gdb) save gdb-index /path/to\ some/directory/

This brings these commands into line with commands like 'file' and
'symbol-file', which have supported quoted filenames for a while.

The motivation for this change is to make handling of filename
arguments consistent throughout GDB.  We can't move to all commands
taking non-quoted filenames as the non-quoted style only allows for a
single argument.  Additionally, the non-quoted style doesn't allow for
filenames that end in white space (though this is probably pretty
rare).  So, if we want to have consistency the only choice is to move
towards supporting quote filenames.

Reviewed-By: Eli Zaretskii <eliz@gnu.org>
gdb/NEWS
gdb/compile/compile.c
gdb/doc/gdb.texinfo
gdb/dwarf2/index-write.c
gdb/target-descriptions.c
gdb/testsuite/gdb.base/filename-completion.exp
gdb/testsuite/gdb.compile/compile.exp