1296742 - clang-cl warning: unintended name hiding in {gfxFont,gfxGDIFont}::GetAdjustedSize

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[Nathan Froyd [:froydnj]]

clang-cl complains thusly:

 3:36.66 In file included from c:/m-c/gfx/thebes/gfxGDIFont.cpp:6:
 3:36.66 c:/m-c/gfx/thebes/gfxGDIFont.h(31,22):  warning: 'gfxGDIFont::GetAdjustedSize' hides overloaded virtual function [-Woverloaded-virtual]
 3:36.66     virtual gfxFloat GetAdjustedSize()
 3:36.66                      ^
 3:36.66 c:/m-c/gfx/thebes/gfxFont.h(1436,22):  note: hidden overloaded virtual function 'gfxFont::GetAdjustedSize' declared here: different qualifiers (const vs none)
 3:36.66     virtual gfxFloat GetAdjustedSize() const {
 3:36.66                      ^

I guess we need to remove the |const| qualifier and add an |override|?

Comment 1

[Jonathan Kew [:jfkthame]]

Attachment: Initialize gfxGDIFont's metrics at construction time, like other platform font subclasses, and remove its no-longer-needed support for init-on-first-use pattern

One solution here would be to just remove 'const' from gfxFont::GetAdjustedSize, but that turns into a bit of a rabbit-hole as we'd then need to de-constify methods that call it, all the way up the call stacks. I think it's cleaner and simpler to make gfxGDIFont initialize its metrics at construction time, like all our other gfxFont subclasses do, and then we can just get rid of its reimplementation of GetAdjustedSize altogether. (Try build with this and bug 1296740: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9ee69da3198b.)

Comment 2

[Jonathan Kew [:jfkthame]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e182a8cc0ed9a61cdf3546f2f175916e2d5aa2c4
Bug 1296742 - Initialize gfxGDIFont's metrics at construction time, like other platform font subclasses, and remove its no-longer-needed support for init-on-first-use pattern. r=jrmuizel

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e182a8cc0ed9