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.tromm...@mailbox.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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to