refactor: clean up serialport.go and serialportlist.go for clarity

Author: dido18

serialport.go

@@ -240,7 +240,10 @@ func (p *serport) writerNoBuf() {
 	p.OnMessage([]byte(msgstr))
 
 	p.portIo.Close()
-	// serialPorts.List(
+
+	// The effect of this call is to send in to all the we client the list of serial ports
+	// TODO: investigate if this is superfluous and it can be removed.
+	// serialPorts.List()
 
 }
 

serialportlist.go

@@ -12,11 +12,10 @@ import (
 )
 
 type serialPortList struct {
-	tools *tools.Tools
-
 	Ports     []*SpPortItem
 	portsLock sync.Mutex
 
+	tools  *tools.Tools `json:"-"`
 	OnList func([]byte) `json:"-"`
 	OnErr  func(string) `json:"-"`
 }