perf ui: No need to set ui_browser to 1 twice
authorArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 13 Aug 2019 15:07:14 +0000 (12:07 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 14 Aug 2019 14:00:00 +0000 (11:00 -0300)
We need to do it only when fallbacking from GTK to the TUI.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: https://lkml.kernel.org/n/tip-dda0acxqef1k72n9z4myjbjt@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/ui/setup.c

index 44fe824e96cd01ab686c4d7c9ee80ca9a4d1cbd6..3bc7c9a6fae9312e66d82e5a038973f6eb4f452a 100644 (file)
@@ -89,9 +89,9 @@ void setup_browser(bool fallback_to_pager)
                printf("GTK browser requested but could not find %s\n",
                       PERF_GTK_DSO);
                sleep(1);
+               use_browser = 1;
                /* fall through */
        case 1:
-               use_browser = 1;
                if (ui__init() == 0)
                        break;
                /* fall through */