Closed Bug 857250 Opened 7 years ago Closed 7 years ago

GCC 4.8 warning: Ion.cpp:321:39: warning: typedef 'MoveOperand' locally defined but not used [-Wunused-local-typedefs]

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: dholbert, Assigned: dholbert)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

js/src/ion/Ion.cpp:321:39: warning: typedef 'MoveOperand' locally defined but not used [-Wunused-local-typedefs]

The warning is for this chunk of code:
318 IonCode *
319 IonCompartment::getVMWrapper(const VMFunction &f)
320 {
321     typedef MoveResolver::MoveOperand MoveOperand;
322 
323     JS_ASSERT(rt->functionWrappers_);
324     JS_ASSERT(rt->functionWrappers_->initialized());
325     IonRuntime::VMWrapperMap::Ptr p = rt->functionWrappers_->readonlyThreadsafeLookup(&f);
326     JS_ASSERT(p);
327 
328     return p->value;
329 }
https://mxr.mozilla.org/mozilla-central/source/js/src/ion/Ion.cpp#318

hg blame says this typedef was added here:
 https://hg.mozilla.org/mozilla-central/rev/6ba78023b367#l2.189
and it looks like it was already unused at the time.  (Probably just copypasted boilerplate or something.)

Anyway, looks like it can be removed.
Comment on attachment 732467 [details] [diff] [review]
fix: remove typedef

Oops, apparently I forgot to request review.
Attachment #732467 - Flags: review?(dvander)
Attachment #732467 - Flags: review?(dvander) → review+
https://hg.mozilla.org/mozilla-central/rev/4b9d6e04de88
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.