Messages in this thread | | | Date | Thu, 05 Jul 2012 10:47:59 -0600 | From | David Ahern <> | Subject | Re: [PATCH] perf tool: save cmdline from user in file header vs what is passed to record |
| |
On 7/5/12 9:48 AM, Stephane Eranian wrote: > On Tue, Jul 3, 2012 at 6:08 AM, David Ahern <dsahern@gmail.com> wrote: > Yes, you need to manually add the pfm_header__set_cmdline(). That begs
I take it you meant perf_header__set_cmdline, not pfm_header...
> the question > as to whether or not it would not be better to move this call directly into > the parse_option() function given that it must be done BEFORE parsing actually > takes place.That way we would be sure all present and future commands would > be covered.
Sure, I'll move it to parse_options.
David
| |