Closed Bug 869730 Opened 12 years ago Closed 11 years ago

Fix an opt build error with ggc enabled

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch v0 (obsolete) — Splinter Review
I missed that JSRuntime::gcDisableStrictProxyCheckingCount is DEBUG only.
Attachment #746691 - Flags: review?(wmccloskey)
Sorry, this is my fault. I changed it recently and didn't change the nursery. I think a better way, if it works, would be to add an AutoDisableProxyCheck field to the MinorCollectionTracer.
Attached patch v1Splinter Review
Thank you, this is much nicer. I also added guard object macros to the guard object, just because.
Attachment #746691 - Attachment is obsolete: true
Attachment #746691 - Flags: review?(wmccloskey)
Attachment #747706 - Flags: review?(wmccloskey)
Attachment #747706 - Flags: review?(wmccloskey) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: