diff options
author | Pedro Alves <palves@redhat.com> | 2019-06-13 00:06:54 +0100 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2019-06-13 00:24:17 +0100 |
commit | 6206060d9be8da3e701fe0307c6c01390e2b4ae2 (patch) | |
tree | 3e8f08e72db035a5278c44f53f737d59d8a547f1 /gdb/cli/cli-utils.c | |
parent | 6665660a411ead049daa85cac5c629d637e22044 (diff) | |
download | binutils-6206060d9be8da3e701fe0307c6c01390e2b4ae2.zip binutils-6206060d9be8da3e701fe0307c6c01390e2b4ae2.tar.gz binutils-6206060d9be8da3e701fe0307c6c01390e2b4ae2.tar.bz2 |
Delete parse_flags/parse_flags_qcs
Now that "thread/frame apply" have been converted to the gdb::option
framework, these functions are no longer used.
For a while, I thought about keeping the unit tests, by making a local
version of parse_flags_qcs in the unit tests file. But all that would
really test that is used by GDB itself, is the validate_flags_qcs
function. So in the end, I went through all the unit tests, and
converted any that wasn't already covered to gdb.base/options.exp
tests. And those have all already been added in previous patches.
gdb/ChangeLog:
2019-06-13 Pedro Alves <palves@redhat.com>
* cli/cli-utils.c (parse_flags, parse_flags_qcs): Delete.
* cli/cli-utils.h (parse_flags, parse_flags_qcs): Delete.
* unittests/cli-utils-selftests.c (test_parse_flags)
(test_parse_flags_qcs): Delete.
(test_cli_utils): Don't call deleted functions.
Diffstat (limited to 'gdb/cli/cli-utils.c')
-rw-r--r-- | gdb/cli/cli-utils.c | 56 |
1 files changed, 0 insertions, 56 deletions
diff --git a/gdb/cli/cli-utils.c b/gdb/cli/cli-utils.c index 30d4091..f5d47ae 100644 --- a/gdb/cli/cli-utils.c +++ b/gdb/cli/cli-utils.c @@ -524,62 +524,6 @@ check_for_argument (const char **str, const char *arg, int arg_len) /* See documentation in cli-utils.h. */ -int -parse_flags (const char **str, const char *flags) -{ - const char *p = skip_spaces (*str); - - if (p[0] == '-' - && isalpha (p[1]) - && (p[2] == '\0' || isspace (p[2]))) - { - const char pf = p[1]; - const char *f = flags; - - while (*f != '\0') - { - if (*f == pf) - { - *str = skip_spaces (p + 2); - return f - flags + 1; - } - f++; - } - } - - return 0; -} - -/* See documentation in cli-utils.h. */ - -bool -parse_flags_qcs (const char *which_command, const char **str, - qcs_flags *flags) -{ - switch (parse_flags (str, "qcs")) - { - case 0: - return false; - case 1: - flags->quiet = true; - break; - case 2: - flags->cont = true; - break; - case 3: - flags->silent = true; - break; - default: - gdb_assert_not_reached ("int qcs flag out of bound"); - } - - validate_flags_qcs (which_command, flags); - - return true; -} - -/* See documentation in cli-utils.h. */ - void validate_flags_qcs (const char *which_command, qcs_flags *flags) { |