Closed Bug 770268 Opened 7 years ago Closed 7 years ago

jsinfer.h:98:24: warning: inline function ‘js::types::TypeObject* js::types::Type::typeObject() const’ used but never defined [enabled by default] (also for "singleObject() const")

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: dholbert, Assigned: luke)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [js:t])

Just noticed that in my (up-to-date mozilla-central debug) build, I'm getting multiple instances of these 2 warnings:
{
In file included from ../../../mozilla/js/src/jsobj.h:23:0,
                 from ../../../mozilla/js/src/jsfun.h:14,
                 from ../../../mozilla/js/src/vm/Stack.h:11,
                 from ../../../mozilla/js/src/jscntxt.h:32,
                 from ../../../mozilla/js/src/frontend/ParseMaps-inl.h:11,
                 from ../../../mozilla/js/src/frontend/ParseMaps.cpp:8:
../../../mozilla/js/src/jsinfer.h:98:24: warning: inline function ‘js::types::TypeObject* js::types::Type::typeObject() const’ used but never defined [enabled by default]
../../../mozilla/js/src/jsinfer.h:90:22: warning: inline function ‘JSObject* js::types::Type::singleObject() const’ used but never defined [enabled by default]
}

Not exactly sure when this warning was introduced, but I'm flagging a dependency on bug 748119 whose cset moded these functions' impls into a different file.

This appears particularly spammy, since it's in a .h file, which means
  (a) we hit it for multiple .cpp files (at least 11)
  (b) it prints out the whole header-includes route from the .cpp file to jsinfer.h, which adds 6 lines of warning-spam in the case above.
(In reply to Daniel Holbert [:dholbert] from comment #0)
> Not exactly sure when this warning was introduced, but I'm flagging a
> dependency on bug 748119 whose cset moded these functions' impls into a
> different file.

(er, s/moded/moved/)
(also: not suggesting that the moving was a bad thing; just that it's one of the factors involved in triggering this warning.)
http://hg.mozilla.org/integration/mozilla-inbound/rev/c9c5ff6c3559
Assignee: general → luke
Target Milestone: --- → mozilla16
Yay!

(/me changes dependency to the bug that added the code fixed by the patch in comment 2)
Blocks: 767141
No longer blocks: 748119
Whiteboard: [js:t]
er, we probably meant to mark this FIXED when the cset comment 2 was merged to m-c. Doing so now.  The m-c cset is: https://hg.mozilla.org/mozilla-central/rev/c9c5ff6c3559
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.