Closed Bug 946791 Opened 6 years ago Closed 6 years ago

GLContextGLX::mLibType is no longer used

Categories

(Core :: Graphics, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla28

People

(Reporter: keeler, Assigned: keeler)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

0:07.50 /home/keeler/mozilla-central/gfx/gl/GLContextProviderGLX.cpp:970:13: error: private field 'mLibType' is not used [-Werror,-Wunused-private-field]

This is with clang. It looks like bug 942501 removed the last instance of its use (as far as I can tell - there may be something more subtle going on): http://hg.mozilla.org/mozilla-central/diff/93d12e87d7d6/gfx/gl/GLContextProviderGLX.cpp#l1.195
Attached patch patchSplinter Review
This fixes it for me, but again, I'm not entirely sure something more subtle isn't going on (e.g. was it intended that this no longer be used?)
Anyway, r? to Benoit.
Attachment #8343192 - Flags: review?(bjacob)
There are more occurences of mLibType in this file than just these two ones, so I am unsure how this patch compiles. Or does it apply to a different tree?

It is true that mLibType is not useful at the moment, and that it would be a good thing to remove it. It was meant to allow switching between system OpenGL, and a custom build of llvmpipe that we'd ship with Firefox as a software fallback for WebGL rendering. That never happened. It's OK to drop this code now. We have a better plan to implement such things going forward anyway, should this topic be brought up again.
Comment on attachment 8343192 [details] [diff] [review]
patch

r- f+ for reasons explained in above comment.
Attachment #8343192 - Flags: review?(bjacob)
Attachment #8343192 - Flags: review-
Attachment #8343192 - Flags: feedback+
(In reply to Benoit Jacob [:bjacob] from comment #2)
> There are more occurences of mLibType in this file than just these two ones,
> so I am unsure how this patch compiles. Or does it apply to a different tree?

All other uses are the mLibType in GLXLibrary. This patch removes the occurrences in GLContextGLX.
This is on mozilla-central.
Comment on attachment 8343192 [details] [diff] [review]
patch

Oh, sorry, I got confused. Thanks.
Attachment #8343192 - Flags: review- → review+
Great - thanks!
(Looking back at that patch, there's not enough context to figure out that GLXLibrary::mLibType is being left alone, so I added that to the commit message.)
https://hg.mozilla.org/integration/mozilla-inbound/rev/6fd483b8cf66
https://hg.mozilla.org/mozilla-central/rev/6fd483b8cf66
Assignee: nobody → dkeeler
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.