Simplify StringTypeAdapter templates
Created attachment 239134 [details] Patch
Comment on attachment 239134 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239134&action=review > Source/WTF/wtf/text/StringConcatenate.h:35 > +#ifndef StringView_h Wut. > Source/WTF/wtf/text/StringView.h:177 > +#include <wtf/text/WTFString.h> Why is this down here? > Source/WTF/wtf/text/StringView.h:179 > +namespace WTF { Why a second namespace WTF?
(In reply to comment #2) > (From update of attachment 239134 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=239134&action=review > > > Source/WTF/wtf/text/StringConcatenate.h:35 > > +#ifndef StringView_h > > Wut. It's an external header guard. > > > Source/WTF/wtf/text/StringView.h:177 > > +#include <wtf/text/WTFString.h> > > Why is this down here? It's to avoid a cyclical include chain. > > > Source/WTF/wtf/text/StringView.h:179 > > +namespace WTF { > > Why a second namespace WTF? We don't want to #include when inside the namespace.
Created attachment 239136 [details] Patch
Created attachment 239143 [details] Patch
Committed r174234: <http://trac.webkit.org/changeset/174234>