On 2014/09/22 15:29:55, Yang wrote:
On 2014/08/19 12:29:00, Yang wrote:
> Please add test cases for cases specific to this impelementation, like
for n
=
> 2^m-1, etc.
>
> Also, please sign the CLA:
> https://developers.google.com/open-source/cla/individual?csw=1
>
>
https://codereview.chromium.org/401783003/diff/80001/src/harmony-string.js
> File src/harmony-string.js (right):
>
>
https://codereview.chromium.org/401783003/diff/80001/src/harmony-string.js#newcode31
> src/harmony-string.js:31: if (n % 2) res += s;
> two-character indent please.
>
>
https://codereview.chromium.org/401783003/diff/80001/src/harmony-string.js#newcode36
> src/harmony-string.js:36: // unroll last iteration, no need to double
the
> initial string again at the
> Capitalize "unroll".
>
>
https://codereview.chromium.org/401783003/diff/80001/src/harmony-string.js#newcode43
> src/harmony-string.js:43: return %StringBuilderConcat(array, 1, "");
> Use %FlattenString
Just to be clear, I followed up on the CLA thing. Everything is fine in
that
regard. I'm just waiting for you to address the remaining issues so that
I can
land this.
Closing this since I landed https://codereview.chromium.org/657863002/
https://codereview.chromium.org/401783003/
--
--
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.
