Profile on start

[bmeck]

simple way to start the cpu profiler when you application starts up using --profile-cpu. useful for applications that need help profiling startup time.

[Qard]

Nice. 👍

[r-52]

I believe you could remove the static const char* - it's just an empty char, that the compiler would replace in the method call.
Another idea: let the user choose the name of the profiler title - the profiler title name could be the next value after your flag and if it isn't set, it becomes an empty string. Just to avoid potential collisions with addons that enable the profiler with the same profiler title name.

[bnoordhuis]

If the title is always empty, you might as well drop this variable and use String::Empty() below.

[bnoordhuis]

Left some comments. I think the help section should make it clear that the flag is not related to --prof, otherwise it's going to be confusing to users.

[Fishrock123]

CI: https://jenkins-iojs.nodesource.com/job/node-test-pull-request/120/

[bmeck]

@bnoordhuis did the slight rephrasing help? will need to rebase/squash before pulling in

[Fishrock123]

@bnoordhuis LGTY?

[bnoordhuis]

Please capitalize and punctuate the comment.

[bnoordhuis]

@bmeck Can you rebase and squash to a single commit with a commit log conforming to the guidelines from CONTRIBUTING.md?

Anyone have thoughts on the name of the switch? I'm partial to --start-cpu-profiler.

[jasnell]

@bmeck @bnoordhuis ... any further updates on this? @bmeck it would need to be rebased and updated before it could land.

[jasnell]

Closing due to lack of activity or response