Closed Bug 901799 Opened 6 years ago Closed 6 years ago

js/src/ion/IonMacroAssembler.cpp:707:20: warning: unused variable 'compartment' [-Wunused-variable]

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: dholbert, Assigned: dholbert)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

New build warning on inbound:
{
js/src/ion/IonMacroAssembler.cpp:707:20: warning: unused variable 'compartment' [-Wunused-variable]
}

This is a variable that bug 898886's checkin today left behind (removed the last usage of), here:
 https://hg.mozilla.org/integration/mozilla-inbound/rev/f836042326f9#l17.41
   17.41      JSCompartment *compartment = GetIonContext()->compartment;
   17.42  
   17.43 -    void *interrupt = (void*)&compartment->rt->interrupt;
   17.44 +    void *interrupt = (void*)&GetIonContext()->runtime->interrupt;

Looks like we can just drop the unused variable now.
Attachment #786094 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/mozilla-central/rev/876c7f8cbe32
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.