WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
189381
Cleanup: Remove extraneous "using namespace" and scope WebCore::LogThreading
https://bugs.webkit.org/show_bug.cgi?id=189381
Summary
Cleanup: Remove extraneous "using namespace" and scope WebCore::LogThreading
Daniel Bates
Reported
2018-09-06 16:01:44 PDT
Today I made some changes in my local working copy that affected the grouping of unified source files such that I got my build failed with the following errors: [[ In file included from /Volumes/.../DerivedSources/WebCore/unified-sources/UnifiedSource29-mm.mm:1: platform/ios/wak/WKGraphics.mm:34:17: error: expected namespace name using namespace WebCore; ^ In file included from /Volumes/.../DerivedSources/WebCore/unified-sources/UnifiedSource29-mm.mm:3: platform/ios/wak/WebCoreThread.mm:868:20: error: no member named 'Threading' in namespace 'WebCore'; did you mean 'LogThreading'? using WebCore::Threading; ~~~~~~~~~^~~~~~~~~ LogThreading In file included from /Volumes/.../DerivedSources/WebCore/unified-sources/UnifiedSource29-mm.mm:3: In file included from platform/ios/wak/WebCoreThread.mm:33: /Volumes/Data/WebKitDev/Safari/OpenSource/Source/WebCore/platform/Logging.h:109:22: note: 'LogThreading' declared here WEBCORE_LOG_CHANNELS(DECLARE_LOG_CHANNEL) ^ 2 errors generated. ]]
Attachments
Patch
(2.08 KB, patch)
2018-09-06 16:05 PDT
,
Daniel Bates
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Daniel Bates
Comment 1
2018-09-06 16:05:18 PDT
Created
attachment 349088
[details]
Patch
Daniel Bates
Comment 2
2018-09-07 16:00:15 PDT
Comment on
attachment 349088
[details]
Patch Clearing flags on attachment: 349088 Committed
r235816
: <
https://trac.webkit.org/changeset/235816
>
Daniel Bates
Comment 3
2018-09-07 16:00:16 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 4
2018-09-07 16:01:26 PDT
<
rdar://problem/44242882
>
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