Code cleanup: Use "new Set()" instead of cloneSet()

RESOLVED FIXED in Firefox 23

Status

--
trivial
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Tracking

Trunk
Firefox 23
All
Windows 8.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 734703 [details] [diff] [review]
patch

I just noticed that the Set() constructor takes an iterable, so we don't need to write our own cloneSet() function that does the same thing.
Attachment #734703 - Flags: review?(sfoster)
Comment on attachment 734703 [details] [diff] [review]
patch

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

Nice, less code. I saw it took an array, shoulda known it was just any iterable.
Attachment #734703 - Flags: review?(sfoster) → review+
https://hg.mozilla.org/mozilla-central/rev/de37ace9e98a
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.