Last Comment Bug 886949 - Add Number.parseInt and Number.parseFloat
: Add Number.parseInt and Number.parseFloat
: dev-doc-complete
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla25
Assigned To: Till Schneidereit [till]
: Jason Orendorff [:jorendorff]
Depends on:
Blocks: es6
  Show dependency treegraph
Reported: 2013-06-25 12:46 PDT by Brandon Benvie [:benvie]
Modified: 2015-02-22 22:48 PST (History)
9 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Add Number.parseInt and Number.parseFloat (9.20 KB, patch)
2013-07-05 13:53 PDT, Till Schneidereit [till]
jwalden+bmo: review+
Details | Diff | Splinter Review

Description User image Brandon Benvie [:benvie] 2013-06-25 12:46:45 PDT
In ES6, `Number.parseInt` and `Number.parseFloat` have been added, and are the same as the global functions of the same names (purpose is modularization of globals).

See ES6 spec (May 2013 revision) sections and
Comment 1 User image Till Schneidereit [till] 2013-07-05 13:53:47 PDT
Created attachment 771686 [details] [diff] [review]
Add Number.parseInt and Number.parseFloat

This is all very straight-forward.

As these are meant to behave identical to their global function counterparts, I just added them to number_static_methods.

In case they ever deviate from the global functions in any way, the duplicated tests should catch any errors that might otherwise sneak in.
Comment 3 User image Ed Morley [:emorley] 2013-07-15 02:57:08 PDT
Comment 4 User image Anthony Hughes (:ashughes) [GFX][QA][Mentor] 2013-09-13 11:21:04 PDT
Assuming this does not need QA. Please remove the [qa-] whiteboard tag and add the verifyme keyword if QA is needed.

Note You need to log in before you can comment on or make changes to this bug.