Added ReentryHelpers to help stop Window:closed event being thrown mu…

…ltiple times

plugins/application/lib/application/window.rb

@@ -178,7 +178,9 @@ def close
       if notebooks.length > 1
         close_notebook
       end
-      notify_listeners(:closed, self)
+      ignore(:closing) do
+        notify_listeners(:closed, self)
+      end
     end
 
     def inspect

plugins/application_swt/lib/application_swt/speedbar.rb

@@ -1,6 +1,8 @@
 module Redcar
   class ApplicationSWT
     class Speedbar
+      include Redcar::ReentryHelpers
+
       attr_reader :widget
 
       def initialize(window, parent, model)
@@ -51,16 +53,6 @@ def focussable_widgets
         @focussable_widgets ||= []
       end
 
-      def ignore_changes
-        @ignore_changes ||= Hash.new(0)
-      end
-
-      def ignore(name)
-        ignore_changes[name] += 1
-        yield if ignore_changes[name] == 1
-        ignore_changes[name] -= 1
-      end
-
       def create_widgets
         create_bar_widget
         create_item_widgets

plugins/core/lib/core.rb

@@ -3,6 +3,7 @@
 require 'fileutils'
 
 require "core/logger"
+require "core/reentry_helpers"
 require "core/controller"
 require "core/gui"
 require "core/interface"

plugins/core/lib/core/model.rb

@@ -1,5 +1,7 @@
 module Redcar
   module Model
+    include Redcar::ReentryHelpers
+
     attr_reader :controller
 
     def controller=(controller)

plugins/core/lib/core/reentry_helpers.rb

@@ -0,0 +1,13 @@
+module Redcar
+  module ReentryHelpers
+    def ignore_changes
+      @ignore_changes ||= Hash.new(0)
+    end
+
+    def ignore(name)
+      ignore_changes[name] += 1
+      yield if ignore_changes[name] == 1
+      ignore_changes[name] -= 1
+    end
+  end
+end