2014-06-03 02:57:45

by Namhyung Kim

[permalink] [raw]
Subject: Re: [PATCH] toosl/perf: convert "==" into "="

Hi Arnaldo,

On Fri, 30 May 2014 12:47:09 -0300, Arnaldo Carvalho de Melo wrote:
> Em Fri, May 30, 2014 at 05:37:29PM +0200, Jiri Olsa escreveu:
>> On Fri, May 30, 2014 at 12:20:11PM -0300, Arnaldo Carvalho de Melo wrote:
>> > Em Fri, May 30, 2014 at 02:44:46PM +0200, Jean Delvare escreveu:
>> > > I don't have anything to do with this, I'm not the author of the code
>> > > nor the maintainer and I don't know anything about it. Arnaldo Carvalho
>> > > de Melo is the right person to handle this bug.
>> >
>> > Jiri,
>
>> > Please take this patch, you can stick my:
>
>> > Acked-by: Arnaldo Carvalho de Melo <[email protected]>
>
>> sure, but I dont see the patch on the lkml.. any chance of resend?
>
>> I guess I could dig the patch from above.. if you confirm it's
>> the only change ;-)
>
> Hey, no need to go to such great lenghts, submitters must try and make
> it easier to maintainers! :-)
>
> Find it attached, zhangdianfang, please send it as an attachment next
> time, also please CC lkml as well.


[SNIP]
>
> convert "==" into "="
>
> Bug description: https://bugzilla.kernel.org/show_bug.cgi?id=76751
>
> Cc: Jean Delvare <[email protected]>
> Reported-by: David Binderman <[email protected]>
> Signed-off-by: Dianfang Zhang <[email protected]>

Acked-by: Namhyung Kim <[email protected]>

Thanks,
Namhyung


> ---
> tools/perf/ui/browser.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c
> index d11541d..3ccf6e1 100644
> --- a/tools/perf/ui/browser.c
> +++ b/tools/perf/ui/browser.c
> @@ -194,7 +194,7 @@ int ui_browser__warning(struct ui_browser *browser, int timeout,
> ui_helpline__vpush(format, args);
> va_end(args);
> } else {
> - while ((key == ui__question_window("Warning!", text,
> + while ((key = ui__question_window("Warning!", text,
> "Press any key...",
> timeout)) == K_RESIZE)
> ui_browser__handle_resize(browser);