Remove break in the switch

Author: mstmdev

auth/perm.go

@@ -81,13 +81,10 @@ func ToPerm(perm string) (p Perm) {
 		switch c {
 		case ReadPerm:
 			r = true
-			break
 		case WritePerm:
 			w = true
-			break
 		case ExecutePerm:
 			x = true
-			break
 		default:
 			return p
 		}

internal/signal/signal.go

@@ -19,15 +19,13 @@ func Notify(shutdown func() error) {
 			err := shutdown()
 			if err != nil {
 				log.Error(err, "shutdown error")
-				break
 			} else {
 				signal.Stop(c)
 				close(c)
 				return
 			}
 		default:
 			log.Debug("received a signal [%s], ignore it", s.String())
-			break
 		}
 	}
 }

monitor/remote_client_monitor.go

@@ -232,16 +232,12 @@ func (m *remoteClientMonitor) parseMessage(data []byte) error {
 	switch status.ApiType {
 	case contract.AuthApi:
 		m.authChan <- status
-		break
 	case contract.InfoApi:
 		m.infoChan <- contract.NewMessage(data)
-		break
 	case contract.SyncMessageApi:
 		m.messages.PushBack(contract.NewMessage(data))
-		break
 	default:
 		log.Warn("remote client monitor received a unknown data => %s", string(data))
-		break
 	}
 	return nil
 }
@@ -290,25 +286,20 @@ func (m *remoteClientMonitor) execSync(msg sync.Message) (err error) {
 	switch msg.Action {
 	case action.CreateAction:
 		err = m.syncer.Create(path)
-		break
 	case action.WriteAction:
 		err = m.syncer.Create(path)
 		// ignore is not exist error
 		if err != nil && os.IsNotExist(err) {
 			err = nil
 		}
 		m.addWrite(path)
-		break
 	case action.RemoveAction:
 		m.removeWrite(path)
 		err = m.syncer.Remove(path)
-		break
 	case action.RenameAction:
 		err = m.syncer.Rename(path)
-		break
 	case action.ChmodAction:
 		err = m.syncer.Chmod(path)
-		break
 	}
 
 	m.el.Write(eventlog.NewEvent(path, msg.Action.String()))

server/handler/push_handler.go

@@ -64,16 +64,12 @@ func (h *pushHandler) Handle(c *gin.Context) {
 	switch pushData.Action {
 	case action.CreateAction:
 		err = h.create(fi)
-		break
 	case action.RemoveAction:
 		err = h.remove(fi)
-		break
 	case action.RenameAction:
 		err = h.rename(fi)
-		break
 	case action.ChmodAction:
 		err = h.chmod(fi)
-		break
 	case action.WriteAction:
 		r, _ := h.write(pushData, c)
 		c.JSON(http.StatusOK, r)

sync/push_client_sync.go

@@ -177,13 +177,10 @@ func (pcs *pushClientSync) receive() {
 				switch status.ApiType {
 				case contract.AuthApi:
 					pcs.authChan <- status
-					break
 				case contract.InfoApi:
 					pcs.infoChan <- contract.NewMessage(data)
-					break
 				default:
 					log.Warn("[push client sync] receive and discard data => %s", string(data))
-					break
 				}
 			}
 		}