LGTM.
http://codereview.chromium.org/109015/diff/1/2 File test/cctest/test-strings.cc (right): http://codereview.chromium.org/109015/diff/1/2#newcode397 Line 397: if (destructed_ != NULL) { Get rid of the NULL checks here. Maybe even CHECK that it isn't NULL. http://codereview.chromium.org/109015/diff/1/2#newcode403 Line 403: if (destructed_ != NULL) { ... and here. http://codereview.chromium.org/109015 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
