diff options
author | Steven Rostedt <srostedt@redhat.com> | 2011-02-14 13:45:51 -0500 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2011-02-14 15:37:06 -0500 |
commit | 00bacb780b8b5076224577ec15be8b4d7d6ec99d (patch) | |
tree | fcbdb3c3ab5c06592169ae244e9779468d47b9dc | |
parent | 9cce1c9b5b58c0236203d1f0b20e8c3d26babe94 (diff) |
trace-cmd/report: Fix switch statement for --filter-test
The long opt for the filter test "--filter-test" was not setting
the filter-test variable. It was not even tested and would cause
the help message to appear.
Also add comments around the long op case statements.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r-- | trace-read.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/trace-read.c b/trace-read.c index d87a89d..15284f5 100644 --- a/trace-read.c +++ b/trace-read.c | |||
@@ -773,15 +773,18 @@ void trace_report (int argc, char **argv) | |||
773 | break; | 773 | break; |
774 | case 0: | 774 | case 0: |
775 | switch(option_index) { | 775 | switch(option_index) { |
776 | case 0: | 776 | case 0: /* cpu */ |
777 | if (filter_cpu) | 777 | if (filter_cpu) |
778 | add_cpu(optarg); | 778 | add_cpu(optarg); |
779 | else | 779 | else |
780 | filter_cpu = atoi(optarg); | 780 | filter_cpu = atoi(optarg); |
781 | break; | 781 | break; |
782 | case 1: | 782 | case 1: /* events */ |
783 | print_events = 1; | 783 | print_events = 1; |
784 | break; | 784 | break; |
785 | case 2: /* filter-test */ | ||
786 | test_filters = 1; | ||
787 | break; | ||
785 | default: | 788 | default: |
786 | usage(argv); | 789 | usage(argv); |
787 | } | 790 | } |