Closed Bug 949747 Opened 7 years ago Closed 7 years ago

Use accessor methods for JSFunction fields

Categories

(Core :: JavaScript Engine, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: bhackett1024, Assigned: bhackett1024)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file)

Attached patch patchSplinter Review
fun->nargs and fun->flags are public fields, and used during Ion compilation.  These should use accessor methods to add locking assertions and mark accesses on them as threadsafe.  The attached patch makes this change, and makes the remaining fields of JSFunction private (the big complicated union was still public though there are no accesses to it outside of JSFunction).
Attachment #8346901 - Flags: review?(jdemooij)
Attachment #8346901 - Flags: review?(jdemooij) → review+
https://hg.mozilla.org/mozilla-central/rev/c22a4358920a
Assignee: nobody → bhackett1024
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.