Don't place "using namespace XXX;" in global space for unified source builds (In reply to Yusuke Suzuki from bug #188589 comment #25) > Maybe, I think we should remove `using namespace XXX` as much as possible > since we are now deploying unified builds. But I think this should be done > in a separate patch. (In reply to Darin Adler from bug #188589 comment #28) > Yes, I think we should move this over time. Unified builds makes it a risky > approach, and I think itβs always been risky from a "reduced clarity" point > of view.
Created attachment 347624 [details] Patch
Created attachment 347625 [details] Patch
Created attachment 347626 [details] Patch
Comment on attachment 347626 [details] Patch Nice! r=me
Comment on attachment 347626 [details] Patch Thank you for the review!
Comment on attachment 347626 [details] Patch Clearing flags on attachment: 347626 Committed r235115: <https://trac.webkit.org/changeset/235115>
All reviewed patches have been landed. Closing bug.
<rdar://problem/43559532>