ES6: Add support for explicit octal and binary integer literals

Add support for explicit octal and binary integer literals

http://people.mozilla.org/~jorendorff/es6-draft.html#sec-7.8.3

ES6 extends the numeric literals to support explicit support
for binary and octal literals using the following syntax:

  0b10101
  0o777

This is currently behind the flag, --harmony-numeric-literals

BUG=2783
R=rossberg@chromium.org

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

[rossberg, 2013-07-16 13:26:25]

Looks good, but should go behind a flag for now. I suggest --harmony-literals.

https://codereview.chromium.org/19300002/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/19300002/diff/1/src/runtime.cc#newcode5958
src/runtime.cc:5958: StringToDouble(isolate->unicode_cache(), subject, ALLOW_HEX
| ALLOW_OCTAL | ALLOW_BINARY));
Nit: line length

[arv (Not doing code reviews), 2013-07-16 14:06:04]

On 2013/07/16 13:26:25, rossberg wrote:
> Looks good, but should go behind a flag for now. I suggest --harmony-literals.

I think --harmony-numeric-literal is better since there are changes to other
literals in ES6 (object literals for example)
 
> https://codereview.chromium.org/19300002/diff/1/src/runtime.cc
> File src/runtime.cc (right):
> 
> https://codereview.chromium.org/19300002/diff/1/src/runtime.cc#newcode5958
> src/runtime.cc:5958: StringToDouble(isolate->unicode_cache(), subject,
ALLOW_HEX
> | ALLOW_OCTAL | ALLOW_BINARY));
> Nit: line length

[arv (Not doing code reviews), 2013-07-16 15:44:12]

PTAL

[arv (Not doing code reviews), 2013-07-17 15:28:32]

Ping?

[rossberg, 2013-07-17 17:05:52]

LGTM, will land tomorrow

[rossberg, 2013-07-18 17:30:54]

Arv, there are plenty of line length violations. Can you please fix those?

[arv (Not doing code reviews), 2013-07-18 17:50:13]

Fixed the long lines.

PTAL

[rossberg, 2013-07-19 09:57:47]

Committed patchset #4 manually as r15772 (presubmit successful).

[rossberg, 2013-07-19 09:58:13]

On 2013/07/18 17:50:13, arv wrote:
> Fixed the long lines.

Thanks, landed.