[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1435655804.18320.15.camel@mailbox.tu-dresden.de>
Date: Tue, 30 Jun 2015 11:16:44 +0200
From: Philipp Trommler <philipp.trommler@...lbox.tu-dresden.de>
To: Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
Thomas Ilsche <thomas.ilsche@...dresden.de>,
Joseph Schuchart <joseph.schuchart@...dresden.de>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] perf: Fixed indentation errors in Core.py
Hi!
This is just a short reminder for the patch I sent a few weeks ago. As
it's just a really small change I'd like to ask if someone could review
it, so either I have the chance to fix errors or it can be applied.
Thanks in advance,
Philipp Trommler.
Am Freitag, den 05.06.2015, 11:01 +0200 schrieb Philipp Trommler:
> Those errors prevented the script from running. Just changed some
> tabs to
> spaces.
>
> Signed-off-by: Philipp Trommler <
> philipp.trommler@...lbox.tu-dresden.de>
> ---
> .../python/Perf-Trace-Util/lib/Perf/Trace/Core.py | 28 +++++++++++-
> ----------
> 1 file changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/tools/perf/scripts/python/Perf-Trace
> -Util/lib/Perf/Trace/Core.py b/tools/perf/scripts/python/Perf-Trace
> -Util/lib/Perf/Trace/Core.py
> index 38dfb72..b4a3857 100644
> --- a/tools/perf/scripts/python/Perf-Trace
> -Util/lib/Perf/Trace/Core.py
> +++ b/tools/perf/scripts/python/Perf-Trace
> -Util/lib/Perf/Trace/Core.py
> @@ -31,7 +31,7 @@ def flag_str(event_name, field_name, value):
> string = ""
>
> if flag_fields[event_name][field_name]:
> - print_delim = 0
> + print_delim = 0
> keys = flag_fields[event_name][field_name]['values'].keys()
> keys.sort()
> for idx in keys:
> @@ -55,10 +55,10 @@ def symbol_str(event_name, field_name, value):
> keys.sort()
> for idx in keys:
> if not value and not idx:
> - string =
> symbolic_fields[event_name][field_name]['values'][idx]
> + string =
> symbolic_fields[event_name][field_name]['values'][idx]
> break
> - if (value == idx):
> - string =
> symbolic_fields[event_name][field_name]['values'][idx]
> + if (value == idx):
> + string =
> symbolic_fields[event_name][field_name]['values'][idx]
> break
>
> return string
> @@ -77,16 +77,16 @@ def trace_flag_str(value):
> keys = trace_flags.keys()
>
> for idx in keys:
> - if not value and not idx:
> - string += "NONE"
> - break
> -
> - if idx and (value & idx) == idx:
> - if print_delim:
> - string += " | ";
> - string += trace_flags[idx]
> - print_delim = 1
> - value &= ~idx
> + if not value and not idx:
> + string += "NONE"
> + break
> +
> + if idx and (value & idx) == idx:
> + if print_delim:
> + string += " | ";
> + string += trace_flags[idx]
> + print_delim = 1
> + value &= ~idx
>
> return string
>
> --
> 2.4.2
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists