<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Mar 31, 2010, at 10:08 AM, Alexey Proskuryakov wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div><div>On 30.03.2010, at 22:44,<span class="Apple-converted-space">&nbsp;</span><a href="mailto:barraclough@apple.com">barraclough@apple.com</a><span class="Apple-converted-space">&nbsp;</span>wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><span class="Apple-style-span" style="font-family: monospace; line-height: 12px; white-space: pre; "><ins style="background-image: initial; background-repeat: initial; background-attachment: initial; -webkit-background-clip: initial; -webkit-background-origin: initial; background-color: rgb(221, 255, 221); text-decoration: none; display: block; padding-top: 0px; padding-right: 10px; padding-bottom: 0px; padding-left: 10px; ">+WebCString::operator WTF::CString() const </ins></span></span></blockquote><br></div><div>There probably should be a "using CString" directive in CString.h - we do that for all public symbols in WTF.</div></span></blockquote><div><br></div><div>There should be ...&nbsp;<a href="http://trac.webkit.org/browser/trunk/JavaScriptCore/wtf/text/CString.h?rev=56825#L79">http://trac.webkit.org/browser/trunk/JavaScriptCore/wtf/text/CString.h?rev=56825#L79</a> ... and there already is!</div><br><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div>The idea is that we don't need or want namespaces to avoid name conflicts between public WTF symbols and WebCore or WebKit, as we control them all.</div></span></blockquote><div><br></div></div>There are two problems that currently cause name conflicts for the name CString in ToT. &nbsp; One is that JSC currently defines a class called CString (when I land&nbsp;<a href="https://bugs.webkit.org/show_bug.cgi?id=36871">https://bugs.webkit.org/show_bug.cgi?id=36871</a> this will go away), so in .cpp files that do a "using namespace JSC" the name CString is ambiguous. &nbsp;Secondly, Windows has a standard type called CString in its libraries, which appears to conflict when compiling some files. &nbsp;Additionally, when changing some files exclusively used in ports that I don't build locally I chose to be conservative &amp; reduce this risk of setting the tree on fire by just replacing WebCore::CString with WTF::CString while landing this patch.<div><br></div><div>Once JSC::CString has gone away I'll have a look through at some point &amp; see if some of the remaining WTF::s are redundant and can be removed.</div><div><br></div><div>cheers,</div><div>G.</div><div><br></div><div><br></div></body></html>