Good point, I didn't see that..

https://codereview.chromium.org/976203003/diff/20001/test/cctest/test-cpu-profiler.cc
File test/cctest/test-cpu-profiler.cc (right):

https://codereview.chromium.org/976203003/diff/20001/test/cctest/test-cpu-profiler.cc#newcode1141
test/cctest/test-cpu-profiler.cc:1141: delete processor;
On 2015/03/05 14:57:40, yurys wrote:
Why not use SmartPointer?

Or even simpler: Just avoid pointers altogether here and simply
construct the object directly without new?

https://codereview.chromium.org/976203003/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to