Closed
Bug 578590
Opened 16 years ago
Closed 15 years ago
extraneous dereference in JSAtomList::rawLookup
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: dherman, Assigned: dherman)
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file)
|
1.72 KB,
patch
|
brendan
:
review+
|
Details | Diff | Splinter Review |
jsatom.cpp, line 1069:
ale = *hep ? (JSAtomListElement *) *hep : NULL;
should be
ale = (JSAtomListElement *) *hep;
Itty bitty patch impending, forthwith. Verily.
Dave
|
Assignee | |
Comment 1•16 years ago
|
||
It ain't rocket science.
Dave
Attachment #457253 -
Flags: review?(brendan)
|
||
Updated•16 years ago
|
Attachment #457253 -
Flags: review?(brendan) → review+
|
|
Assignee | |
Comment 2•16 years ago
|
||
Pushed.
Dave
|
||
Comment 3•16 years ago
|
||
We prefer changeset URLs when marking as pushed, since Mercurial makes it so easy to get a nice change view. :-) (Some people used to post screenfuls of |cvs commit| output, when we used CVS. It was pretty awful, and doing so was far less common as a result.) And make sure to flag as fixed-in-tracemonkey if that's where you pushed a fix.
http://hg.mozilla.org/tracemonkey/rev/207f3548ff02
Status: NEW → ASSIGNED
Whiteboard: fixed-in-tracemonkey
|
|
Assignee | |
Comment 4•16 years ago
|
||
/me nods. Thanks for the info. I'll get it right next time. :)
Dave
|
||
Comment 5•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•