* Andrea Gelmini <andrea.gelmini@xxxxxxxxx> wrote: > tools/perf/builtin-trace.c: util/exec_cmd.h and util/trace-event.h are included more than once. > > Signed-off-by: Andrea Gelmini <andrea.gelmini@xxxxxxxxx> > --- > tools/perf/builtin-trace.c | 3 --- > 1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c > index 407041d..3c699c5 100644 > --- a/tools/perf/builtin-trace.c > +++ b/tools/perf/builtin-trace.c > @@ -59,9 +59,6 @@ static int cleanup_scripting(void) > #include "perf.h" > #include "util/debug.h" > > -#include "util/trace-event.h" > -#include "util/exec_cmd.h" > - > static char const *input_name = "perf.data"; > > static int process_sample_event(event_t *event, struct perf_session *session) Please send a more complete patch that fixes the root cause of the duplication: a few header files are in the middle of the .c file - all inclusions should be at the top of the file. Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html