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 45893
Add the namespace in a friend declaration to get MSVC recognize it properly.
https://bugs.webkit.org/show_bug.cgi?id=45893
Summary
Add the namespace in a friend declaration to get MSVC recognize it properly.
Satish Sampath
Reported
2010-09-16 08:26:07 PDT
This is a fix for a build break caused by
r67614
in the chromium windows builders (which use VS 2008). Looks like without the proper namespace, the friend declaration was treated as a forward declaration and the build failed with the following error message: 1>JavaScriptCore\wtf/RefCounted.h(139) : error C2248: 'WebCore::FileWriter::~FileWriter' : cannot access private member declared in class 'WebCore::FileWriter' 1> webcore\fileapi\FileWriter.h(124) : compiler has generated 'WebCore::FileWriter::~FileWriter' here 1> webcore\fileapi\FileWriter.h(52) : see declaration of 'WebCore::FileWriter' 1> JavaScriptCore\wtf/RefCounted.h(137) : while compiling class template member function 'void WTF::RefCounted<T>::deref(void)' 1> with 1> [ 1> T=WebCore::FileWriter 1> ] 1> webcore\fileapi\FileWriter.h(52) : see reference to class template instantiation 'WTF::RefCounted<T>' being compiled 1> with 1> [ 1> T=WebCore::FileWriter 1> ]
Attachments
Patch
(1.02 KB, patch)
2010-09-16 08:28 PDT
,
Satish Sampath
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Satish Sampath
Comment 1
2010-09-16 08:28:24 PDT
Created
attachment 67800
[details]
Patch
Jeremy Orlow
Comment 2
2010-09-16 08:33:01 PDT
Comment on
attachment 67800
[details]
Patch r=me
Satish Sampath
Comment 3
2010-09-16 08:40:27 PDT
Comment on
attachment 67800
[details]
Patch Clearing flags on attachment: 67800 Committed
r67622
: <
http://trac.webkit.org/changeset/67622
>
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