Closed Bug 1296273 Opened 6 years ago Closed 6 years ago

Remove subarray and copyWithin from vm/TypedArrayCommon.h

Categories

(Core :: JavaScript: Standard Library, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox51 --- affected
firefox52 --- fixed

People

(Reporter: anba, Assigned: anba)

Details

Attachments

(1 file)

The subarray and copyWithin methods in vm/TypedArrayCommon.h don't appear to be used anymore (cf. bug 1176214), so we should probably remove them.

[1] http://hg.mozilla.org/mozilla-central/file/97a52326b06a/js/src/vm/TypedArrayCommon.h#l611
[2] http://hg.mozilla.org/mozilla-central/file/97a52326b06a/js/src/vm/TypedArrayCommon.h#l703
Attached patch bug1296273.patchSplinter Review
The C++ implementations of subarray() and copyWithin() are no longer used.
Assignee: nobody → andrebargull
Status: NEW → ASSIGNED
Attachment #8802262 - Flags: review?(lhansen)
Comment on attachment 8802262 [details] [diff] [review]
bug1296273.patch

Review of attachment 8802262 [details] [diff] [review]:
-----------------------------------------------------------------

The smaller TypedArrayCommon.h becomes, the better, in my opinion...
Attachment #8802262 - Flags: review?(lhansen) → review+
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c9ddce297cab
Remove no longer used native implementation of TypedArray methods. r=lth
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c9ddce297cab
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.