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
Comment on attachment 557328 [details] [diff] [review] fix: remove unused variable Yep. Mental flotsam.