Re: [PATCH 5/7] perf top: Delay UI browser setup after initialization is done

2015-12-10 Thread Arnaldo Carvalho de Melo
Em Thu, Dec 10, 2015 at 02:52:05PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Thu, Dec 10, 2015 at 02:43:32PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Thu, Dec 10, 2015 at 12:00:57PM +0900, Namhyung Kim escreveu: > > > Move setup_browser after all necessary initialization is done. This

Re: [PATCH 5/7] perf top: Delay UI browser setup after initialization is done

2015-12-10 Thread Arnaldo Carvalho de Melo
Em Thu, Dec 10, 2015 at 02:43:32PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Thu, Dec 10, 2015 at 12:00:57PM +0900, Namhyung Kim escreveu: > > Move setup_browser after all necessary initialization is done. This > > is to remove the browser dependency from usage_with_options() and > > friends.

Re: [PATCH 5/7] perf top: Delay UI browser setup after initialization is done

2015-12-10 Thread Arnaldo Carvalho de Melo
Em Thu, Dec 10, 2015 at 12:00:57PM +0900, Namhyung Kim escreveu: > Move setup_browser after all necessary initialization is done. This > is to remove the browser dependency from usage_with_options() and > friends. So, please try: perf top -C 0 -p 1 So that we get a command line validation error

[PATCH 5/7] perf top: Delay UI browser setup after initialization is done

2015-12-09 Thread Namhyung Kim
Move setup_browser after all necessary initialization is done. This is to remove the browser dependency from usage_with_options() and friends. Cc: Josh Poimboeuf Signed-off-by: Namhyung Kim --- tools/perf/builtin-top.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff