Skip to contentSkip to conversation

perf ui: Make --stdio default when TUI is not supported

tip:perf/urgent tip-bot for Namhyung Kim 1 participant 1 message May 30, 2012 1 message not in the archive perf events

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

#1 lore raw
Commit-ID:  21f0d423b989284f82004d0f80c75581fec91fb2
Gitweb:     http://git.kernel.org/tip/21f0d423b989284f82004d0f80c75581fec91fb2
Author:     Namhyung Kim <[email protected]>
AuthorDate: Mon, 28 May 2012 23:53:22 +0900
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Mon, 28 May 2012 13:29:54 -0300

perf ui: Make --stdio default when TUI is not supported

The commit dc41b9b8f02db ("perf ui: Change fallback policy of setup_browser") changed default behavior of the function but missed setting the use_browser variable to 0 accidently. So perf report ends up doing nothing in such cases. Fix it.

---
 tools/perf/ui/setup.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
tools/perf/ui/setup.c +1 -0
index 9f5f888..791fb15 100644
@@ -22,6 +22,7 @@ void setup_browser(bool fallback_to_pager)
 			break;
 		/* fall through */
 	default:
+		use_browser = 0;
 		if (fallback_to_pager)
 			setup_pager();
 		break;