namespace {} trick for SK_DECLARE_STATIC_ONCE

src/utils/win/SkDWrite.cpp

Index: src/utils/win/SkDWrite.cpp
diff --git a/src/utils/win/SkDWrite.cpp b/src/utils/win/SkDWrite.cpp
index 7801059187c0907cba9dac5b01afd6df2f7bd8b1..363ac438d99334bf4287ca100325d9a208678108 100644
--- a/src/utils/win/SkDWrite.cpp
+++ b/src/utils/win/SkDWrite.cpp
@@ -41,10 +41,9 @@ static void create_dwrite_factory(IDWriteFactory** factory) {
 }
 
 
+SK_DECLARE_STATIC_ONCE(once);

[bungeman-skia, 2015/01/12 19:36:37]

'once' seems like a rather generic thing to put at global scope like this, but I
don't see any good reason to change it... ('gOnce' isn't really any better). It
was quite nice when these were scoped.

 IDWriteFactory* sk_get_dwrite_factory() {
-    SK_DECLARE_STATIC_ONCE(once);
     SkOnce(&once, create_dwrite_factory, &gDWriteFactory);
-
     return gDWriteFactory;
 }