Style comments first: please Start comments with capital letter and end with period.
http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc File test/cctest/test-api.cc (right): http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1307 test/cctest/test-api.cc:1307: const AccessorInfo& info) { indention http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1313 test/cctest/test-api.cc:1313: const AccessorInfo& info) { indention and put value on seperate line http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1319 test/cctest/test-api.cc:1319: const AccessorInfo& info) { indention http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1331 test/cctest/test-api.cc:1331: // intercept names that start with 'interceptor_' Capitalize I at start of comment http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1331 test/cctest/test-api.cc:1331: // intercept names that start with 'interceptor_' period at the end of comment http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1336 test/cctest/test-api.cc:1336: for (i = 0; name_str[i] && prefix[i]; ++i) Please add {} around body when not on single line http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1345 test/cctest/test-api.cc:1345: // intercept accesses that set certain integer values Capital I at start of comment, period at the end http://codereview.chromium.org/7991007/diff/1/test/cctest/test-api.cc#newcode1362 test/cctest/test-api.cc:1362: v8::NamedPropertyGetter getter, Indention http://codereview.chromium.org/7991007/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
