diff options
| author | Andrew Burgess <aburgess@redhat.com> | 2024-06-20 13:44:28 +0100 |
|---|---|---|
| committer | Andrew Burgess <aburgess@redhat.com> | 2024-09-07 20:28:59 +0100 |
| commit | e454ae416a050496107f08cb0ae0de518d732a90 (patch) | |
| tree | 13622c82f13832412cf8de08c02ff55f28960b1f /gdb/NEWS | |
| parent | d552429edae9a9cb6d5c6506149fff1aa407daeb (diff) | |
| download | fsf-binutils-gdb-e454ae416a050496107f08cb0ae0de518d732a90.zip fsf-binutils-gdb-e454ae416a050496107f08cb0ae0de518d732a90.tar.gz fsf-binutils-gdb-e454ae416a050496107f08cb0ae0de518d732a90.tar.bz2 | |
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>
Diffstat (limited to 'gdb/NEWS')
| -rw-r--r-- | gdb/NEWS | 7 |
1 files changed, 7 insertions, 0 deletions
@@ -63,6 +63,13 @@ remove-symbol-file -a ADDRESS multiple terms, e.g. 'function + 0x1000' (without quotes), previously only a single term could be given. +compile file +maint print c-tdesc +save gdb-index + These commands now require their filename argument to be quoted if + it contains white space or quote characters. If the argument + contains no such special characters then quoting is not required. + * New remote packets vFile:stat |