removed nil channel sets

ste/jobStatusManager.go

@@ -176,19 +176,17 @@ func (jm *jobMgr) handleStatusUpdateMessage() {
 		case <-jstm.listReq:
 			/* Display stats */
 			js.Timestamp = time.Now().UTC()
-			if jstm.respChan != nil {
-				select {
-				case jstm.respChan <- *js:
-					// Send on the channel
-				case <-jstm.statusMgrDone:
-					// If we time out, no biggie. This isn't world-ending, nor is it essential info. The other side stopped listening by now.
-				}
-				defer func() { // Exit gracefully if panic
-					if recErr := recover(); recErr != nil {
-						jm.Log(common.LogError, "Cannot send message on respChan")
-					}
-				}()
+			select {
+			case jstm.respChan <- *js:
+				// Send on the channel
+			case <-jstm.statusMgrDone:
+				// If we time out, no biggie. This isn't world-ending, nor is it essential info. The other side stopped listening by now.
 			}
+			defer func() { // Exit gracefully if panic
+				if recErr := recover(); recErr != nil {
+					jm.Log(common.LogError, "Cannot send message on respChan")
+				}
+			}()
 			// Reset the lists so that they don't keep accumulating and take up excessive memory
 			// There is no need to keep sending the same items over and over again
 			js.FailedTransfers = []common.TransferDetail{}
@@ -198,8 +196,6 @@ func (jm *jobMgr) handleStatusUpdateMessage() {
 				close(jstm.statusMgrDone)
 				close(jstm.respChan)
 				close(jstm.listReq)
-				jstm.listReq = nil
-				jstm.respChan = nil
 				return
 			}
 		}