perf events parse: Remove some needless local variables
authorArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 17 Aug 2017 18:57:30 +0000 (15:57 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 17 Aug 2017 19:39:15 +0000 (16:39 -0300)
Those are just casting a void pointer to a struct to then pass them to
functions, i.e. remove the local variables and pass the void pointer
directly, the casting will be done and the code will be shorter.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-bzfodzr3mb46gy7u7v0mqad6@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/parse-events.y

index 04fd8c9af9f9578c0f8f44a3c8cb05cf087d992f..96c0fa59aa8f4f49bfc85ac2db75c314258b06c5 100644 (file)
@@ -225,14 +225,13 @@ event_def: event_pmu |
 event_pmu:
 PE_NAME opt_event_config
 {
-       struct parse_events_evlist *data = _data;
        struct list_head *list, *orig_terms, *terms;
 
        if (parse_events_copy_term_list($2, &orig_terms))
                YYABORT;
 
        ALLOC_LIST(list);
-       if (parse_events_add_pmu(data, list, $1, $2)) {
+       if (parse_events_add_pmu(_data, list, $1, $2)) {
                struct perf_pmu *pmu = NULL;
                int ok = 0;
 
@@ -245,7 +244,7 @@ PE_NAME opt_event_config
                        if (!strncmp($1, name, strlen($1))) {
                                if (parse_events_copy_term_list(orig_terms, &terms))
                                        YYABORT;
-                               if (!parse_events_add_pmu(data, list, pmu->name, terms))
+                               if (!parse_events_add_pmu(_data, list, pmu->name, terms))
                                        ok++;
                                parse_events_terms__delete(terms);
                        }
@@ -286,26 +285,24 @@ PE_VALUE_SYM_SW
 event_legacy_symbol:
 value_sym '/' event_config '/'
 {
-       struct parse_events_evlist *data = _data;
        struct list_head *list;
        int type = $1 >> 16;
        int config = $1 & 255;
 
        ALLOC_LIST(list);
-       ABORT_ON(parse_events_add_numeric(data, list, type, config, $3));
+       ABORT_ON(parse_events_add_numeric(_data, list, type, config, $3));
        parse_events_terms__delete($3);
        $$ = list;
 }
 |
 value_sym sep_slash_dc
 {
-       struct parse_events_evlist *data = _data;
        struct list_head *list;
        int type = $1 >> 16;
        int config = $1 & 255;
 
        ALLOC_LIST(list);
-       ABORT_ON(parse_events_add_numeric(data, list, type, config, NULL));
+       ABORT_ON(parse_events_add_numeric(_data, list, type, config, NULL));
        $$ = list;
 }
 
@@ -432,11 +429,10 @@ PE_NAME ':' PE_NAME
 event_legacy_numeric:
 PE_VALUE ':' PE_VALUE opt_event_config
 {
-       struct parse_events_evlist *data = _data;
        struct list_head *list;
 
        ALLOC_LIST(list);
-       ABORT_ON(parse_events_add_numeric(data, list, (u32)$1, $3, $4));
+       ABORT_ON(parse_events_add_numeric(_data, list, (u32)$1, $3, $4));
        parse_events_terms__delete($4);
        $$ = list;
 }
@@ -444,11 +440,10 @@ PE_VALUE ':' PE_VALUE opt_event_config
 event_legacy_raw:
 PE_RAW opt_event_config
 {
-       struct parse_events_evlist *data = _data;
        struct list_head *list;
 
        ALLOC_LIST(list);
-       ABORT_ON(parse_events_add_numeric(data, list, PERF_TYPE_RAW, $1, $2));
+       ABORT_ON(parse_events_add_numeric(_data, list, PERF_TYPE_RAW, $1, $2));
        parse_events_terms__delete($2);
        $$ = list;
 }
@@ -468,11 +463,10 @@ PE_BPF_OBJECT opt_event_config
 |
 PE_BPF_SOURCE opt_event_config
 {
-       struct parse_events_evlist *data = _data;
        struct list_head *list;
 
        ALLOC_LIST(list);
-       ABORT_ON(parse_events_load_bpf(data, list, $1, true, $2));
+       ABORT_ON(parse_events_load_bpf(_data, list, $1, true, $2));
        parse_events_terms__delete($2);
        $$ = list;
 }