Closed Bug 495330 Opened 11 years ago Closed 10 years ago

Trace JSOP_INCNAME/JSOP_NAMEINC for closures

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dmandelin, Assigned: dmandelin)

References

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file, 1 obsolete file)

See bug 492918.
Attached patch Patch (obsolete) — Splinter Review
I think it's done, but I want to let the try server go over it before asking for review.
Attached patch Patch 2Splinter Review
Attachment #391256 - Attachment is obsolete: true
Attachment #391676 - Flags: review?(gal)
Comment on attachment 391676 [details] [diff] [review]
Patch 2

|tracked| looks weird. I think we usually use '"tracked"' but I am not sure. I like NameResult. This is going to conflict the hell out of jorendorff's patch in my queue, so adding him.
Attachment #391676 - Flags: review?(gal) → review+
Pushed to TM as e3d2c39d5aa4.

I left |tracked| in because we have stuff like |this| in there already. But if you really don't like it, I won't mind at all if you change it.
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/e3d2c39d5aa4
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.