From 558e5987be5258ab8faaf4dd05e126966f969e7f Mon Sep 17 00:00:00 2001
From: Jacob Heun <jacobheun@gmail.com>
Date: Tue, 5 Feb 2019 15:26:55 +0100
Subject: [PATCH] chore: remove _isStarted as it's no longer used

_isStarted is an outdated property and shouldn't exist
in the code anymore. The state machine handles start
logic and the isStarted computed property pulls from there.
The code comments around dht and pubsub are also no
longer valid, so they were deleted.
---
 src/index.js | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/index.js b/src/index.js
index 986e9532..b7e4bb05 100644
--- a/src/index.js
+++ b/src/index.js
@@ -53,7 +53,6 @@ class Node extends EventEmitter {
 
     this._modules = _options.modules
     this._config = _options.config
-    this._isStarted = false
     this._transport = [] // Transport instances/references
     this._discovery = [] // Discovery service instances/references
 
@@ -400,9 +399,6 @@ class Node extends EventEmitter {
         }
       },
       (cb) => {
-        // TODO: chicken-and-egg problem #1:
-        // have to set started here because DHT requires libp2p is already started
-        this._isStarted = true
         if (this._dht) {
           this._dht.start(cb)
         } else {
@@ -410,8 +406,6 @@ class Node extends EventEmitter {
         }
       },
       (cb) => {
-        // TODO: chicken-and-egg problem #2:
-        // have to set started here because FloodSub requires libp2p is already started
         if (this._floodSub) {
           return this._floodSub.start(cb)
         }