Address CR Feedback.

microsoft · Nov 4, 2016 · 8aa78df · 8aa78df
1 parent 5f79b50
commit 8aa78df
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 12 deletions.
diff --git a/Frameworks/CoreGraphics/CGContext.mm b/Frameworks/CoreGraphics/CGContext.mm
@@ -1648,9 +1648,9 @@ void CGContextFillRects(CGContextRef context, const CGRect* rects, size_t count)
 void CGContextDrawPath(CGContextRef context, CGPathDrawingMode mode) {
     NOISY_RETURN_IF_NULL(context);
     if (context->HasPath()) {
-        ID2D1Geometry* pGeometry;
+        ComPtr<ID2D1Geometry> pGeometry;
         FAIL_FAST_IF_FAILED(_CGPathGetGeometry(context->Path(), &pGeometry));
-        __CGContextDrawGeometry(context, pGeometry, mode);
+        FAIL_FAST_IF_FAILED(__CGContextDrawGeometry(context, pGeometry.Get(), mode));
         context->ClearPath();
     }
 }

diff --git a/Frameworks/CoreGraphics/CGPath.mm b/Frameworks/CoreGraphics/CGPath.mm
@@ -152,7 +152,7 @@ HRESULT InitializeGeometries() {
 HRESULT _CGPathGetGeometry(CGPathRef path, ID2D1Geometry** pGeometry) {
     if (path && pGeometry) {
         RETURN_IF_FAILED(path->ClosePath());
-        *pGeometry = path->GetPathGeometry().Get();
+        path->GetPathGeometry().CopyTo(pGeometry);
     }
     return S_OK;
 }

diff --git a/Frameworks/CoreGraphics/D2DWrapper.mm b/Frameworks/CoreGraphics/D2DWrapper.mm
@@ -19,22 +19,16 @@
 
 using namespace Microsoft::WRL;
 
-// Private helper for creating a D2DFactory
 static HRESULT __createD2DFactory(ID2D1Factory** factory) {
     return D2D1CreateFactory(D2D1_FACTORY_TYPE_SINGLE_THREADED, factory);
 }
 
 HRESULT _CGGetD2DFactory(ID2D1Factory** factory) {
     static ComPtr<ID2D1Factory> sFactory;
-    static HRESULT sHr;
-    static dispatch_once_t dispatchToken;
+    static HRESULT sHr = __createD2DFactory(&sFactory);
 
-    dispatch_once(&dispatchToken, ^{
-        sHr = __createD2DFactory(&sFactory);
-    });
-    sFactory.Get()->AddRef();
-    *factory = sFactory.Get();
-    return sHr;
+    sFactory.CopyTo(factory);
+    RETURN_HR(sHr);
 }
 
 static ComPtr<IWICImagingFactory> __createWICFactory() {