loopd: atomic var to enforce daemon singelton

Author: hieblmi

loopd/daemon.go

@@ -70,8 +70,7 @@ type Daemon struct {
 	// same process.
 	swapClientServer
 
-	// To be used atomically.
-	started int32
+	started atomic.Bool
 
 	// ErrChan is an error channel that users of the Daemon struct must use
 	// to detect runtime errors and also whether a shutdown is fully
@@ -128,8 +127,10 @@ func (d *Daemon) Start() error {
 	// There should be no reason to start the daemon twice. Therefore,
 	// return an error if that's tried. This is mostly to guard against
 	// Start and StartAsSubserver both being called.
-	if atomic.AddInt32(&d.started, 1) != 1 {
+	if d.started.Load() {
 		return errOnlyStartOnce
+	} else {
+		d.started.Store(true)
 	}
 
 	network := lndclient.Network(d.cfg.Network)
@@ -190,8 +191,10 @@ func (d *Daemon) StartAsSubserver(lndGrpc *lndclient.GrpcLndServices,
 	// There should be no reason to start the daemon twice. Therefore,
 	// return an error if that's tried. This is mostly to guard against
 	// Start and StartAsSubserver both being called.
-	if atomic.AddInt32(&d.started, 1) != 1 {
+	if d.started.Load() {
 		return errOnlyStartOnce
+	} else {
+		d.started.Store(true)
 	}
 
 	// When starting as a subserver, we get passed in an already established