https://codereview.chromium.org/12089107/diff/7001/test/cctest/test-platform.cc
File test/cctest/test-platform.cc (right):
https://codereview.chromium.org/12089107/diff/7001/test/cctest/test-platform.cc#newcode1
test/cctest/test-platform.cc:1: // Copyright 2006-2008 the V8 project
authors. All rights reserved.
On 2013/02/15 10:36:45, Michael Starzinger wrote:
Copyright header should say just "2013", no year-range. Also I would
include the
full copyright header to new files. See e.g. test-api.cc for an
example.
Done.
https://codereview.chromium.org/12089107/diff/7001/test/cctest/test-platform.cc#newcode9
test/cctest/test-platform.cc:9: #include "platform.h"
On 2013/02/15 10:36:45, Michael Starzinger wrote:
Can we alpha-sort the includes?
Done.
https://codereview.chromium.org/12089107/diff/7001/test/cctest/test-platform.cc#newcode9
test/cctest/test-platform.cc:9: #include "platform.h"
On 2013/02/15 10:36:45, Michael Starzinger wrote:
Can we alpha-sort the includes?
Done.
https://codereview.chromium.org/12089107/
--
--
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/groups/opt_out.
