Filing bug on this build warning: > js/src/vm/Debugger.cpp: In static member function ‘static JSTrapStatus js::Debugger::onTrap(JSContext*, js::Value*)’: > js/src/vm/Debugger.cpp:874:11: warning: variable ‘frame’ set but not used [-Wunused-but-set-variable] The variable is declared like so... > 874 Value frame = UndefinedValue(); https://mxr.mozilla.org/mozilla-central/source/js/src/vm/Debugger.cpp#874 ...and is never used after that. (That's actually been the case since its first checkin: https://hg.mozilla.org/mozilla-central/rev/45f1cf2c59d2#l63.306 )
Created attachment 557328 [details] [diff] [review] fix: remove unused variable
Attachment #557328 - Flags: review?
Attachment #557328 - Flags: review? → review?(jorendorff)
Comment on attachment 557328 [details] [diff] [review] fix: remove unused variable Yep. Mental flotsam.
Attachment #557328 - Flags: review?(jorendorff) → review+
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.