drive-by:

http://codereview.chromium.org/7891005/diff/1/src/d8.h
File src/d8.h (right):

http://codereview.chromium.org/7891005/diff/1/src/d8.h#newcode197
src/d8.h:197: if (parallel_files != NULL) delete[] parallel_files;
Same here.

http://codereview.chromium.org/7891005/diff/1/src/d8.h#newcode199
src/d8.h:199: if (isolate_sources != NULL) delete[] isolate_sources;
No need to check for NULL for delete.

http://codereview.chromium.org/7891005/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to