FIX: Handle properly stopping the NSApp when a tooltip panel might still be active

cimarronm · cimarronm · commit 5ed6e2f46bbc · 2017-05-26T17:45:15.000-07:00
diff --git a/src/_macosx.m b/src/_macosx.m
@@ -13,9 +13,6 @@
 
 /* Proper way to check for the OS X version we are compiling for, from
    http://developer.apple.com/documentation/DeveloperTools/Conceptual/cross_development */
-#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1050
-#define COMPILING_FOR_10_5
-#endif
 #if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1060
 #define COMPILING_FOR_10_6
 #endif
@@ -26,13 +23,6 @@
 #define COMPILING_FOR_10_10
 #endif
 
-/* Use Atsui for Mac OS X 10.4, CoreText for Mac OS X 10.5 */
-#ifndef COMPILING_FOR_10_5
-static int ngc = 0;    /* The number of graphics contexts in use */
-
-#include <Carbon/Carbon.h>
-
-#endif
 
 /* CGFloat was defined in Mac OS X 10.5 */
 #ifndef CGFLOAT_DEFINED
@@ -44,6 +34,11 @@
 #define STOP_EVENT_LOOP 2
 #define WINDOW_CLOSING 3
 
+
+/* Keep track of number of windows present
+   Needed to know when to stop the NSApp */
+static long FigureWindowCount = 0;
+
 /* -------------------------- Helper function ---------------------------- */
 
 static void
@@ -664,6 +659,7 @@ static CGFloat _get_device_scale(CGContextRef cr)
 FigureManager_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
 {
     Window* window = [Window alloc];
+    ++FigureWindowCount;
     if (!window) return NULL;
     FigureManager *self = (FigureManager*)type->tp_alloc(type, 0);
     if (!self)
@@ -2022,8 +2018,8 @@ - (BOOL)closeButtonPressed
 - (void)close
 {
     [super close];
-    NSArray *windowsArray = [NSApp windows];
-    if([windowsArray count]==0) [NSApp stop: self];
+    --FigureWindowCount;
+    if (!FigureWindowCount) [NSApp stop: self];
     /* This is needed for show(), which should exit from [NSApp run]
      * after all windows are closed.
      */
