[gdb/cli] Add "set logging enabled", deprecate "set logging on/off"
authorTom de Vries <tdevries@suse.de>
Thu, 25 Nov 2021 06:49:16 +0000 (07:49 +0100)
committerTom de Vries <tdevries@suse.de>
Thu, 25 Nov 2021 06:49:16 +0000 (07:49 +0100)
commit6ff9675421809af862db5b86d3756ddf4aef568f
treebde802e8c32117bb05e220012683003dd473fca9
parent0a4a1c6a3b46731c228431d5bd29181e0d2107c2
[gdb/cli] Add "set logging enabled", deprecate "set logging on/off"

Before commit 3b6acaee895 "Update more calls to add_prefix_cmd" we had the
following output for "show logging file":
...
$ gdb -q -batch -ex "set trace-commands on" \
    -ex "set logging off" \
    -ex "show logging file" \
    -ex "set logging on" \
    -ex "show logging file"
+set logging off
+show logging file
Future logs will be written to gdb.txt.
+set logging on
+show logging file
Currently logging to "gdb.txt".
...

After that commit we have instead:
...
+set logging off
+show logging file
The current logfile is "gdb.txt".
+set logging on
+show logging file
The current logfile is "gdb.txt".
...

Before the commit, whether logging is enabled or not can be deduced from the
output of the command.  After the commit, the message is unified and it's no
longer clear whether logging is enabled or not.

Fix this by:
- adding a new command "show logging enabled"
- adding a corresponding new command "set logging enabled on/off"
- making the commands "set logging on/off" deprecated aliases of the
  "set logging enabled on/off" command.

Update the docs and testsuite to use "set logging enabled".  Mention the new
and deprecated commands in NEWS.

Tested on x86_64-linux.
gdb/NEWS
gdb/cli/cli-logging.c
gdb/doc/gdb.texinfo
gdb/testsuite/gdb.ada/access_to_packed_array.exp
gdb/testsuite/gdb.base/style-logging.exp
gdb/testsuite/gdb.base/ui-redirect.exp
gdb/testsuite/gdb.mi/mi-logging.exp
gdb/testsuite/lib/gdb.exp