perf tools: Allow '--inherit' as the negation of '--no-inherit'

Long options can be negated by prefixing them with 'no-'.  However
options that already start with 'no-', such as '--no-inherit' result in
ugly double 'no's.

Avoid that by accepting that the removal of 'no-' also negates the long
option.

Suggested-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Namhyung Kim <namhyung@gmail.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1384768557-23331-3-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

1 files changed, 19 insertions, 0 deletions

diff --git a/tools/perf/util/parse-options.c b/tools/perf/util/parse-options.c
index 31f404a032a9..b6b39ff14fbb 100644
--- a/tools/perf/util/parse-options.c
+++ b/tools/perf/util/parse-options.c
@@ -224,6 +224,24 @@ static int parse_long_opt(struct parse_opt_ctx_t *p, const char *arg,
                         return 0;
                 }
                 if (!rest) {
+                        if (!prefixcmp(options->long_name, "no-")) {
+                                /*
+                                 * The long name itself starts with "no-", so
+                                 * accept the option without "no-" so that users
+                                 * do not have to enter "no-no-" to get the
+                                 * negation.
+                                 */
+                                rest = skip_prefix(arg, options->long_name + 3);
+                                if (rest) {
+                                        flags |= OPT_UNSET;
+                                        goto match;
+                                }
+                                /* Abbreviated case */
+                                if (!prefixcmp(options->long_name + 3, arg)) {
+                                        flags |= OPT_UNSET;
+                                        goto is_abbreviated;
+                                }
+                        }
                         /* abbreviated? */
                         if (!strncmp(options->long_name, arg, arg_end - arg)) {
 is_abbreviated:
@@ -259,6 +277,7 @@ is_abbreviated:
                         if (!rest)
                                 continue;
                 }
+match:
                 if (*rest) {
                         if (*rest != '=')
                                 continue;