perf tui: Fix last use_browser problem related to .perfconfig
Arnaldo Carvalho de Melo [Thu, 27 May 2010 12:53:40 +0000 (09:53 -0300)]
When we moved to using ~/.perfconfig to set the value of use_browser,
it changed from a boolean to an int so that the convention used for
use_pager was followed.

That convention is:

-1: unspecified, that is what use_{browser,pager} is initialized
 0: Don't use the browser (should be TUI), because was explicitely
    set to 0/off/false on ~/.perfconfig [tui] cmd =, or because
    we're redirecting the stdout to a file or piping it to some
    other command (!isatty()).
 1: Use the TUI

Some code was not properly audited and continued testing it as a
boolean, this seems to be the last one.

Reported-by: Frédéric Weisbecker <>
Tested-by: Frédéric Weisbecker <>
Cc: Frédéric Weisbecker <>
Cc: Mike Galbraith <>
Cc: Paul Mackerras <>
Cc: Peter Zijlstra <>
Cc: Stephane Eranian <>
Cc: Tom Zanussi <>
LKML-Reference: <new-submission>
Signed-off-by: Arnaldo Carvalho de Melo <>


index dd824cf..6cddff2 100644 (file)
@@ -22,7 +22,7 @@ int eprintf(int level, const char *fmt, ...)
        if (verbose >= level) {
                va_start(args, fmt);
-               if (use_browser)
+               if (use_browser > 0)
                        ret = browser__show_help(fmt, args);
                        ret = vfprintf(stderr, fmt, args);