Moved one check outside of the loop and moved another one under an if stateme...

Moved one check outside of the loop and moved another one under an if statement 

Committed: http://code.google.com/p/v8/source/detail?r=4082

[Erik Corry, 2010-03-10 11:16:43]

You need to install cpplint.py (it's Open Source) and run tools/presubmit.py.  I
think it will complain about tabs (replace with 8 spaces), trailing spaces
(remove) and long lines (max 80).

LGTM

http://codereview.chromium.org/821001/diff/1/2
File src/string.js (right):

http://codereview.chromium.org/821001/diff/1/2#newcode392
src/string.js:392: if (NUMBER_OF_CAPTURES(matchInfo) == 2) {
Please add a comment here to say that the loop is substantially the same in both
branches of the loop for speed.

[sandholm, 2010-03-10 11:59:25]

http://codereview.chromium.org/821001/diff/1/2
File src/string.js (right):

http://codereview.chromium.org/821001/diff/1/2#newcode392
src/string.js:392: if (NUMBER_OF_CAPTURES(matchInfo) == 2) {
On 2010/03/10 11:16:43, Erik Corry wrote:
> Please add a comment here to say that the loop is substantially the same in
both
> branches of the loop for speed.

Done.