Closed Bug 1209325 Opened 9 years ago Closed 9 years ago

js::WeakMapObject should have its own source file, like js/src/builtin/WeakSetObject.{h,cpp}.

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: jimb, Assigned: jimb)

Details

Attachments

(1 file)

The implementation of WeakMap should be in its own source files in js/src/builtins, like WeakSetObject and MapObject.x
Attachment #8666985 - Flags: review?(terrence)
Comment on attachment 8666985 [details] [diff] [review]
Move implementation of JS WeakMap type into its own file.

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

Wow, it's just amazing the things you get used to.
Attachment #8666985 - Flags: review?(terrence) → review+
Flags: in-testsuite-
Target Milestone: --- → mozilla44
https://hg.mozilla.org/mozilla-central/rev/31392a3bfe2a
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: