Closed
Bug 1122913
Opened 9 years ago
Closed 9 years ago
Remove object out-parameter from native LookupOwnProperty
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: evilpie, Assigned: evilpie)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
9.56 KB,
patch
|
jorendorff
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #8550716 -
Flags: review?(jorendorff)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → evilpies
Comment 1•9 years ago
|
||
Comment on attachment 8550716 [details] [diff] [review] remove-objp Review of attachment 8550716 [details] [diff] [review]: ----------------------------------------------------------------- Looks great! ::: js/src/jsobj.cpp @@ +2933,5 @@ > if (!done) { > objp.set(nullptr); > propp.set(nullptr); > return true; > + } else { Style nit: No else after return. ::: js/src/vm/NativeObject-inl.h @@ +427,3 @@ > * cx->resolvingTable, control must go through cleanup: before > * returning. But note that JS_DHASH_ADD may find an existing > * entry, in which case we bail to suppress runaway recursion. While you're fixing comments, note that there is no `cleanup:` label anymore. The whole second paragraph here can be removed, I think.
Attachment #8550716 -
Flags: review?(jorendorff) → review+
Assignee | ||
Comment 2•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d98309db7e20
https://hg.mozilla.org/mozilla-central/rev/d98309db7e20
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in
before you can comment on or make changes to this bug.
Description
•