https://codereview.chromium.org/1102523003/diff/140001/src/preparser.cc
File src/preparser.cc (right):
https://codereview.chromium.org/1102523003/diff/140001/src/preparser.cc#newcode261
src/preparser.cc:261: if (maybe_reset && (!starts_with_identifier ||
++count_statements > 200)) {
On 2015/04/30 13:20:18, jochen wrote:
can we define 200 has a const?
Done.
https://codereview.chromium.org/1102523003/diff/140001/src/scanner-character-streams.cc
File src/scanner-character-streams.cc (right):
https://codereview.chromium.org/1102523003/diff/140001/src/scanner-character-streams.cc#newcode149
src/scanner-character-streams.cc:149: pos_ = bookmark_;
On 2015/04/30 13:20:18, jochen wrote:
why no DCHECK here?
Done.
No big reason, really. For whatever reason I here decided to initialize
the bookmark with 'start_position', so there was never an 'illegal'
value in it. This way, it puts more effort into checking the contract.
https://codereview.chromium.org/1102523003/diff/140001/src/scanner.h
File src/scanner.h (right):
https://codereview.chromium.org/1102523003/diff/140001/src/scanner.h#newcode374
src/scanner.h:374: };
On 2015/04/30 13:20:18, jochen wrote:
disallow copy/assign
Done.
https://codereview.chromium.org/1102523003/diff/140001/src/scanner.h#newcode758
src/scanner.h:758: // - bookmark_c0_ == -1: No bookmark has been set.
On 2015/04/30 13:20:18, jochen wrote:
how about defining an enum for -1 and -2?
Done.
https://codereview.chromium.org/1102523003/
--
--
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.
