Closed Bug 831023 Opened 8 years ago Closed 8 years ago
On non-Mac platforms: gfx/layers/opengl/Canvas
Layer OGL .cpp:89:1: warning: 'GLuint Make IOSurface Texture(void*, mozilla::gl::GLContext*)' defined but not used [-Wunused-function]
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.
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+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.