Re: [PATCH v3 5/5] perf annotate browser: Circulate percent, total period and samples view

From: Arnaldo Carvalho de Melo
Date: Mon Aug 21 2017 - 15:19:45 EST


Em Mon, Aug 21, 2017 at 02:47:09PM +0900, Taeung Song escreveu:
>
>
> On 08/18/2017 11:23 PM, Arnaldo Carvalho de Melo wrote:
> > Em Fri, Aug 18, 2017 at 05:47:08PM +0900, Taeung Song escreveu:
> > > With a existing 't' hotkey, support the three view based on percent,
> > > total period and number of samples on the annotate TUI browser,
> > > circulating them like below:
> > >
> > > Percent -> Period -> Samples -> Percent ...
> > >
> > > Suggested-by: Namhyung Kim <namhyung@xxxxxxxxxx>
> > > Cc: Milian Wolff <milian.wolff@xxxxxxxx>
> > > Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
> > > Signed-off-by: Taeung Song <treeze.taeung@xxxxxxxxx>
> > > ---
> >
> > Ok, here I removed this part, that is not documented in the patch nor in
> > the 'h' help screen, if you think it should be considered, please
> > resubmit it with a proper explanation:
>
> I'm really sorry. The case 'e' code is a residue..
> I missed removing the code.
> Thank you for indicating my mistakes.
>
> Do I resend this patchkit based on your changes ?
> Or, will you modify it by yourself ?

I fixed it myself, sending to Ingo now.

Thanks!

- Arnaldo