WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 62084
clang build error: hidden overloaded virtual function
https://bugs.webkit.org/show_bug.cgi?id=62084
Summary
clang build error: hidden overloaded virtual function
Joseph Pecoraro
Reported
2011-06-03 21:50:21 PDT
This seemed to cause a clang build warning (which is treated as an error).
> CompileC RenderTable.o > In file included from /Users/pecoraro/Code/webkit-open-source/Source/WebCore/rendering/RenderTable.cpp:38: > Source/WebCore/rendering/RenderTableCell.h:146:18: error: 'WebCore::RenderTableCell::paintMask' hides overloaded virtual function [-Werror,-Woverloaded-virtual,3] > Source/WebCore/rendering/RenderBox.h:357:18: note: hidden overloaded virtual function 'WebCore::RenderBox::paintMask' declared here [3] > 1 error generated. > Command /Developer/usr/bin/clang failed with exit code 1
Indeed the functions have a similar but different signature: RenderBox.h: virtual void paintMask(PaintInfo&, const IntPoint&); RenderTableCell.h: virtual void paintMask(PaintInfo&, IntSize); RenderTableCell just takes the incoming IntSize and makes an IntPoint out of it anyways.
Attachments
[PATCH] Build Fix
(2.42 KB, patch)
2011-06-03 22:01 PDT
,
Joseph Pecoraro
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Joseph Pecoraro
Comment 1
2011-06-03 21:51:10 PDT
I meant to say this was caused by: <
http://trac.webkit.org/changeset/88087
> But we should probably just fix RenderTableCell.
Joseph Pecoraro
Comment 2
2011-06-03 22:01:02 PDT
Created
attachment 96007
[details]
[PATCH] Build Fix I just want a sanity check from someone on this one, but I'm pretty sure its the right fix.
Joseph Pecoraro
Comment 3
2011-06-03 22:03:18 PDT
Mitz "The Master" landed this already in: <
http://trac.webkit.org/changeset/88090
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug