chore: merge subs add in Minimap

Author: aminya

lib/minimap.js

@@ -234,49 +234,54 @@ export default class Minimap {
     const subs = this.subscriptions
     let configSubscription = this.subscribeToConfig()
 
-    subs.add(configSubscription)
-
-    subs.add(this.textEditor.onDidChangeGrammar(() => {
-      subs.remove(configSubscription)
-      configSubscription.dispose()
-
-      configSubscription = this.subscribeToConfig()
-      subs.add(configSubscription)
-    }))
+    subs.add(
+      configSubscription,
+
+      this.textEditor.onDidChangeGrammar(() => {
+        subs.remove(configSubscription)
+        configSubscription.dispose()
+
+        configSubscription = this.subscribeToConfig()
+        subs.add(configSubscription)
+      }),
+
+      this.adapter.onDidChangeScrollTop(() => {
+        if (!this.standAlone && !this.ignoreTextEditorScroll && !this.inChangeScrollTop) {
+          this.inChangeScrollTop = true
+          this.updateScrollTop()
+          this.emitter.emit('did-change-scroll-top', this)
+          this.inChangeScrollTop = false
+        }
 
-    subs.add(this.adapter.onDidChangeScrollTop(() => {
-      if (!this.standAlone && !this.ignoreTextEditorScroll && !this.inChangeScrollTop) {
-        this.inChangeScrollTop = true
-        this.updateScrollTop()
-        this.emitter.emit('did-change-scroll-top', this)
-        this.inChangeScrollTop = false
-      }
+        if (this.ignoreTextEditorScroll) {
+          this.ignoreTextEditorScroll = false
+        }
+      }),
 
-      if (this.ignoreTextEditorScroll) {
-        this.ignoreTextEditorScroll = false
-      }
-    }))
-    subs.add(this.adapter.onDidChangeScrollLeft(() => {
-      if (!this.standAlone) {
-        this.emitter.emit('did-change-scroll-left', this)
-      }
-    }))
+      this.adapter.onDidChangeScrollLeft(() => {
+        if (!this.standAlone) {
+          this.emitter.emit('did-change-scroll-left', this)
+        }
+      }),
+
+      this.textEditor.onDidChange((changes) => {
+        this.scheduleChanges(changes)
+      }),
+
+      this.textEditor.onDidDestroy(() => { this.destroy() }),
+
+      /*
+      FIXME Some changes occuring during the tokenization produces
+      ranges that deceive the canvas rendering by making some
+      lines at the end of the buffer intact while they are in fact not,
+      resulting in extra lines appearing at the end of the minimap.
+      Forcing a whole repaint to fix that bug is suboptimal but works.
+      */
+      this.textEditor.onDidTokenize(() => {
+        this.emitter.emit('did-change-config')
+      })
 
-    subs.add(this.textEditor.onDidChange((changes) => {
-      this.scheduleChanges(changes)
-    }))
-    subs.add(this.textEditor.onDidDestroy(() => { this.destroy() }))
-
-    /*
-    FIXME Some changes occuring during the tokenization produces
-    ranges that deceive the canvas rendering by making some
-    lines at the end of the buffer intact while they are in fact not,
-    resulting in extra lines appearing at the end of the minimap.
-    Forcing a whole repaint to fix that bug is suboptimal but works.
-    */
-    subs.add(this.textEditor.onDidTokenize(() => {
-      this.emitter.emit('did-change-config')
-    }))
+    )
   }
 
   /**