Closed
Bug 1492872
Opened 6 years ago
Closed 6 years ago
Remove JSProto_Iterator
Categories
(Core :: JavaScript Engine, enhancement)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: jorendorff, Assigned: jorendorff)
Details
Attachments
(1 file)
It is apparently completely unused.
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
I'm also willing to update the comment at the top of ProtoKey.h that talks about "binary XDR compatibility"... but I'm not sure what it should say. Neither `JSProto_` nor `ProtoKey` appears at all in XDR.cpp these days. Just delete it? Can we get rid of `imaginary` prototypes altogether?
Flags: needinfo?(jwalden+bmo)
Comment 3•6 years ago
|
||
Comment on attachment 9010666 [details] Bug 1492872 - Remove JSProto_Iterator. r?jwalden Jeff Walden [:Waldo] has approved the revision.
Attachment #9010666 -
Flags: review+
Comment 4•6 years ago
|
||
XDR versioning is now on the embedding to rev build id all the time, so there's no XDR concern any more now as far as JSProtoKey goes. (JSProtoKey *does* show up in bytecode in certain places, tho, I think, so if we still had the same bytecode version bumping as of old, it would be be troublesome to just change or remove at least some of these.) We can change the comment. Imaginary protos we've slowly been replacing with one-off slots in GlobalObject. If we have indeed replaced all of them, we should remove imaginariness.
Flags: needinfo?(jwalden+bmo)
Pushed by jorendorff@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/caccc6d0cd9a Remove JSProto_Iterator. r=jwalden
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/caccc6d0cd9a
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Updated•6 years ago
|
Assignee: nobody → jorendorff
You need to log in
before you can comment on or make changes to this bug.
Description
•