Note: There are a few cases of duplicates in user autocompletion which are being worked on.

GCC 4.6 build warning: "js/src/vm/Debugger.cpp:874:11: warning: variable ‘frame’ set but not used [-Wunused-but-set-variable]"

RESOLVED FIXED in mozilla9

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks: 1 bug)

Trunk
mozilla9
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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 )
(Assignee)

Comment 1

6 years ago
Created attachment 557328 [details] [diff] [review]
fix: remove unused variable
Attachment #557328 - Flags: review?
(Assignee)

Updated

6 years ago
Attachment #557328 - Flags: review? → review?(jorendorff)
(Assignee)

Updated

6 years ago
Assignee: general → dholbert
Comment on attachment 557328 [details] [diff] [review]
fix: remove unused variable

Yep. Mental flotsam.
Attachment #557328 - Flags: review?(jorendorff) → review+
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/dc12ae87f5b9
Status: NEW → ASSIGNED
Whiteboard: [inbound]

Comment 4

6 years ago
http://hg.mozilla.org/mozilla-central/rev/dc12ae87f5b9
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.