The default bug view has changed. See this FAQ.

js/src/vm/ObjectImpl-inl.h:214:1: warning: unused function 'IsValueInCompartment' [-Wunused-function]

RESOLVED FIXED in mozilla18

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: gps, Assigned: terrence)

Tracking

Trunk
mozilla18
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
I pulled m-c for the first time in about a week this morning and got a new compiler warning:

js/src/vm/ObjectImpl-inl.h:214:1: warning: unused function 'IsValueInCompartment' [-Wunused-function]

I'm seeing it repeated a few dozen times (because the header is included many places, I assume).

I'm building with Clang SVN HEAD minus a week or two on OS X.
(Assignee)

Comment 1

5 years ago
Looks like I forgot #ifdef DEBUG markers around it.  Thanks for filing.
Assignee: general → terrence
Blocks: 776583
Status: NEW → ASSIGNED
(Assignee)

Comment 2

5 years ago
Created attachment 655628 [details] [diff] [review]
v0: trivial
Attachment #655628 - Flags: review?(gps)
(Reporter)

Comment 3

5 years ago
Comment on attachment 655628 [details] [diff] [review]
v0: trivial

Review of attachment 655628 [details] [diff] [review]:
-----------------------------------------------------------------

I'm not a peer for the JS module, so I don't think I can give r+. It looks good to me though.
Attachment #655628 - Flags: review?(gps) → feedback+
(Assignee)

Comment 4

5 years ago
I consider it a build issue, not a JS issue as such, but whatever.
(Assignee)

Comment 5

5 years ago
Comment on attachment 655628 [details] [diff] [review]
v0: trivial

r+ from bhackett on IRC with feedback from Waldo that I should make it inline instead of static, since inline implies static in C++.
Attachment #655628 - Flags: review+
(Assignee)

Comment 6

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/72bb1083fbab
(Assignee)

Updated

5 years ago
Duplicate of this bug: 785589
https://hg.mozilla.org/mozilla-central/rev/72bb1083fbab
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.