Clear decorations when SearchResults is destroyed

avendael · Jan 27, 2019 · 8fbf9b8 · 8fbf9b8
1 parent 6695a05
commit 8fbf9b8
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/lib/search-results.coffee b/lib/search-results.coffee
@@ -16,6 +16,7 @@ class SearchResults
     @currentDecorations = []
 
   clear: ->
+    @_clearDecorations()
     @markerLayer.clear()
     @_numMatches = 0
 
@@ -45,10 +46,13 @@ class SearchResults
 
   setCurrent: (markers) ->
     # TODO: don't destroy markers that don't need to be destroyed?
-    @currentDecorations.forEach (decoration) ->
-      decoration.destroy()
+    @_clearDecorations()
 
     @currentDecorations = markers.map (marker) =>
       @editor.decorateMarker marker,
         type: 'highlight'
         class: 'atomic-emacs-current-result'
+
+  _clearDecorations: ->
+    @currentDecorations.forEach (decoration) ->
+      decoration.destroy()