From 4770e668116b9809f0ba3d9f29625401790d33fa Mon Sep 17 00:00:00 2001
From: Michael Maltese <michaeljosephmaltese@gmail.com>
Date: Sat, 15 Apr 2017 20:43:22 -0700
Subject: [PATCH] AGL: refactor some functions

---
 Source/Core/Common/GL/GLInterface/AGL.mm | 92 ++++++++++++------------
 1 file changed, 47 insertions(+), 45 deletions(-)

diff --git a/Source/Core/Common/GL/GLInterface/AGL.mm b/Source/Core/Common/GL/GLInterface/AGL.mm
index 594a854f31..51d00ad04d 100644
--- a/Source/Core/Common/GL/GLInterface/AGL.mm
+++ b/Source/Core/Common/GL/GLInterface/AGL.mm
@@ -5,6 +5,45 @@
 #include "Common/GL/GLInterface/AGL.h"
 #include "Common/Logging/Log.h"
 
+static bool UpdateCachedDimensions(NSView* view, u32* width, u32* height)
+{
+  NSWindow* window = [view window];
+  NSSize size = [view frame].size;
+
+  float scale = [window backingScaleFactor];
+  size.width *= scale;
+  size.height *= scale;
+
+  if (*width == size.width && *height == size.height)
+    return false;
+
+  *width = size.width;
+  *height = size.height;
+
+  return true;
+}
+
+static bool AttachContextToView(NSOpenGLContext* context, NSView* view, u32* width, u32* height)
+{
+  // Enable high-resolution display support.
+  [view setWantsBestResolutionOpenGLSurface:YES];
+
+  NSWindow* window = [view window];
+  if (window == nil)
+  {
+    ERROR_LOG(VIDEO, "failed to get NSWindow");
+    return false;
+  }
+
+  (void)UpdateCachedDimensions(view, width, height);
+
+  [window makeFirstResponder:view];
+  [context setView:view];
+  [window makeKeyAndOrderFront:nil];
+
+  return true;
+}
+
 void cInterfaceAGL::Swap()
 {
   [cocoaCtx flushBuffer];
@@ -33,36 +72,11 @@ bool cInterfaceAGL::Create(void* window_handle, bool core)
     return false;
   }
 
-  if (window_handle)
-  {
-    cocoaWin = reinterpret_cast<NSView*>(window_handle);
-    NSSize size = [cocoaWin frame].size;
+  if (!window_handle)
+    return true;
 
-    // Enable high-resolution display support.
-    [cocoaWin setWantsBestResolutionOpenGLSurface:YES];
-
-    NSWindow* window = [cocoaWin window];
-
-    float scale = [window backingScaleFactor];
-    size.width *= scale;
-    size.height *= scale;
-
-    // Control window size and picture scaling
-    s_backbuffer_width = size.width;
-    s_backbuffer_height = size.height;
-
-    if (cocoaWin == nil)
-    {
-      ERROR_LOG(VIDEO, "failed to create window");
-      return false;
-    }
-
-    [window makeFirstResponder:cocoaWin];
-    [cocoaCtx setView:cocoaWin];
-    [window makeKeyAndOrderFront:nil];
-  }
-
-  return true;
+  cocoaWin = static_cast<NSView*>(window_handle);
+  return AttachContextToView(cocoaCtx, cocoaWin, &s_backbuffer_width, &s_backbuffer_height);
 }
 
 bool cInterfaceAGL::MakeCurrent()
@@ -87,23 +101,11 @@ void cInterfaceAGL::Shutdown()
 
 void cInterfaceAGL::Update()
 {
-  if (cocoaWin)
-  {
-    NSWindow* window = [cocoaWin window];
-    NSSize size = [cocoaWin frame].size;
+  if (!cocoaWin)
+    return;
 
-    float scale = [window backingScaleFactor];
-    size.width *= scale;
-    size.height *= scale;
-
-    if (s_backbuffer_width == size.width && s_backbuffer_height == size.height)
-      return;
-
-    s_backbuffer_width = size.width;
-    s_backbuffer_height = size.height;
-  }
-
-  [cocoaCtx update];
+  if (UpdateCachedDimensions(cocoaWin, &s_backbuffer_width, &s_backbuffer_height))
+    [cocoaCtx update];
 }
 
 void cInterfaceAGL::SwapInterval(int interval)