From adcce9475b696b2e002bf9fb13ed3afc2d2b58c6 Mon Sep 17 00:00:00 2001
From: Paul McCarthy <pauldmccarthy@gmail.com>
Date: Mon, 21 Oct 2019 22:29:00 +0100
Subject: [PATCH] DOC: Clarify comments about haveGui situation [skip ci]

---
 fsl/utils/platform.py | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/fsl/utils/platform.py b/fsl/utils/platform.py
index 1001363dc..4a97325bc 100644
--- a/fsl/utils/platform.py
+++ b/fsl/utils/platform.py
@@ -176,19 +176,23 @@ class Platform(notifier.Notifier):
             #      to check that the wx main loop
             #      was running. But this doesn't
             #      suit situations where a non-main
-            #      event loop is running (e.g. when
+            #      event loop is running, or where
+            #      the mainloop is periodically
+            #      started and stopped (e.g. when
             #      the event loop is being run by
             #      IPython).
             #
             #      In c++ wx, there is the
             #      wx.App.UsesEventLoop method, but
             #      this is not presently exposed to
-            #      Python code.
+            #      Python code (and wouldn't help
+            #      to detect the loop start/stop
+            #      scenario).
             #
             #      So this constraint has been
             #      (hopefully) temporarily relaxed
-            #      until UsesEventLoop can be called
-            #      from Python.
+            #      until I can think of a better
+            #      solution.
             return (self.canHaveGui and
                     app is not None)
 
-- 
GitLab