Move Git crash check earlier

problemgenerator · problemgenerator · commit d97a2d8a0440 · 2023-04-09T20:22:11.000+10:00
This check doesn't rely on anything set earlier in `revealLater()`, so
there's no reason for it to be nested within the later `if shouldReveal`
condition.
diff --git a/SyncedSideBar.py b/SyncedSideBar.py
@@ -107,6 +107,11 @@ def show_view(view):
     win = view.window()
 
     def revealLater():
+        # Some versions of Sublime Text crash when revealing files under `.git/` in the side bar:
+        # https://github.com/sublimehq/sublime_text/issues/5881
+        if int(sublime.version()) < 4148 and '/.git/' in str(win.active_view().file_name()):
+            return
+
         if (int(sublime.version()) >= 3098):
             # API provided by sublime
             shouldReveal = win.is_sidebar_visible()
@@ -115,11 +120,6 @@ def revealLater():
             shouldReveal = sidebarVisible
 
         if shouldReveal and reveal != False:
-            # Some versions of Sublime Text crash when revealing files under `.git/` in the side bar:
-            # https://github.com/sublimehq/sublime_text/issues/5881
-            if int(sublime.version()) < 4148 and '/.git/' in str(win.active_view().file_name()):
-                return
-
             win.run_command('reveal_in_side_bar')
 
     # When using quick switch project, the view activates before the sidebar is ready.
