If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

unused variable warnings in Runtime.cpp in optimized builds

RESOLVED FIXED in mozilla26

Status

()

Core
JavaScript Engine
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Benjamin, Assigned: Benjamin)

Tracking

unspecified
mozilla26
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 792988 [details] [diff] [review]
fix it

/dev/mozilla/inbound/js/src/vm/Runtime.cpp
/home/benjamin/dev/mozilla/inbound/js/src/vm/Runtime.cpp: In function ‘bool js::CurrentThreadCanAccessRuntime(JSRuntime*)’:
/home/benjamin/dev/mozilla/inbound/js/src/vm/Runtime.cpp:747:20: warning: unused variable ‘pt’ [-Wunused-variable]
/home/benjamin/dev/mozilla/inbound/js/src/vm/Runtime.cpp: In function ‘bool js::CurrentThreadCanAccessZone(JS::Zone*)’:
/home/benjamin/dev/mozilla/inbound/js/src/vm/Runtime.cpp:755:20: warning: unused variable ‘pt’ [-Wunused-variable]
(Assignee)

Updated

4 years ago
Attachment #792988 - Flags: review?(luke)

Comment 1

4 years ago
Comment on attachment 792988 [details] [diff] [review]
fix it

And a good day to you, sir.
Attachment #792988 - Flags: review?(luke) → review+
(Assignee)

Comment 2

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/be5a8c4d2d21
Assignee: general → benjamin

Comment 3

4 years ago
https://hg.mozilla.org/mozilla-central/rev/be5a8c4d2d21
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.