Improvements to CPU profiler#73
Improvements to CPU profiler#73OleksandrChekhovskyi wants to merge 2 commits intonode-inspector:masterfrom
Conversation
|
Some builds are failing because of errors downloading node.js releases, not because of problems with patches. |
|
@OleksandrChekhovskyi , thank you for contribution. |
Now idle time is marked as "(idle)" instead of just "(program)".
415b1fd to
422f53d
Compare
|
I have updated the first patch, because I've noticed a compiler warning in CI builds. Also see my replies to inline comments. Edit: For some reason, GitHub does not show 'commented on outdated diff' in this PR. Did you comment not from PR? It looks like it just lost the comments after force push, unfortunately. |
|
Reasonable comments. |
Expose more useful data from node.js