On non-Mac platforms: gfx/layers/opengl/CanvasLayerOGL.cpp:89:1: warning: 'GLuint MakeIOSurfaceTexture(void*, mozilla::gl::GLContext*)' defined but not used [-Wunused-function]

RESOLVED FIXED in mozilla21

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks 1 bug)

Trunk
mozilla21
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Build warning:
gfx/layers/opengl/gfx/layers/opengl/CanvasLayerOGL.cpp:89:1: warning: 'GLuint MakeIOSurfaceTexture(void*, mozilla::gl::GLContext*)' defined but not used [-Wunused-function]

This function has an interesting definition in an #ifdef XP_MAC clause, and then a trivial impl in an #else clause.  And then it's only called once, in some #ifdef XP_MAC code.

Two easy solutions: 
  (1) Drop the whole #else clause, since it's unused
 ...or...
  (2) Declare the function as inline instead of static. It's only called once, so inlining it shouldn't be expensive. (and inline functions don't trigger -Wunused-function.)


Note that prior to bug 724666, this function was completely unused. (as reported in bug 797616 and bug 786057)  Now it's just unused-on-non-mac-platforms.
Posted patch fix v1Splinter Review
I'm leaning towards just removing the non-mac version of this function, because it's just a NS_RuntimeAbort anyway. (and it's probably better to catch non-Mac-platform usages of this function at compile time rather than at run time.)
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #702539 - Flags: review?(bgirard)
Comment on attachment 702539 [details] [diff] [review]
fix v1

That works, thanks
Attachment #702539 - Flags: review?(bgirard) → review+
https://hg.mozilla.org/mozilla-central/rev/bb97ca612605
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.