Closed Bug 897875 Opened 6 years ago Closed 6 years ago

Add IsCallPC()

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: efaust, Assigned: efaust)

Details

Attachments

(1 file)

Attached patch isCallPC.patchSplinter Review
Dotted around the codebase are js_CodeSpec[*pc].format & JOF_INVOKE. It's much clearer to read IsCallPC(pc). This is also in line with IsGetterPC() and IsSetterPC().
Attachment #780846 - Flags: review?(hv1989)
Comment on attachment 780846 [details] [diff] [review]
isCallPC.patch

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

Nice work!
Attachment #780846 - Flags: review?(hv1989) → review+
https://hg.mozilla.org/mozilla-central/rev/fb1a587df374
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.