Merge pull request kubernetes#68047 from niuzhenguo/remove-switch-break

k8s-ci-robot · web-flow · commit f50734399b82 · 2019-02-11T21:43:09.000-08:00
Clean up unneeded break in switch
diff --git a/staging/src/k8s.io/apiserver/pkg/endpoints/installer.go b/staging/src/k8s.io/apiserver/pkg/endpoints/installer.go
@@ -411,7 +411,6 @@ func (a *APIInstaller) registerResourceHandlers(path string, storage rest.Storag
 		actions = appendIf(actions, action{"WATCH", "watch/" + itemPath, nameParams, namer, false}, isWatcher)
 		actions = appendIf(actions, action{"CONNECT", itemPath, nameParams, namer, false}, isConnecter)
 		actions = appendIf(actions, action{"CONNECT", itemPath + "/{path:*}", proxyParams, namer, false}, isConnecter && connectSubpath)
-		break
 	default:
 		namespaceParamName := "namespaces"
 		// Handler for standard REST verbs (GET, PUT, POST and DELETE).
@@ -467,7 +466,6 @@ func (a *APIInstaller) registerResourceHandlers(path string, storage rest.Storag
 			// DEPRECATED in 1.11
 			actions = appendIf(actions, action{"WATCHLIST", "watch/" + resource, params, namer, true}, allowWatchList)
 		}
-		break
 	}
 
 	// Create Routes for the actions.

Original file line number	Diff line number	Diff line change
`@@ -411,7 +411,6 @@ func (a *APIInstaller) registerResourceHandlers(path string, storage rest.Storag`
`411`	`411`	`actions = appendIf(actions, action{"WATCH", "watch/" + itemPath, nameParams, namer, false}, isWatcher)`
`412`	`412`	`actions = appendIf(actions, action{"CONNECT", itemPath, nameParams, namer, false}, isConnecter)`
`413`	`413`	`actions = appendIf(actions, action{"CONNECT", itemPath + "/{path:*}", proxyParams, namer, false}, isConnecter && connectSubpath)`
`414`		`- break`
`415`	`414`	`default:`
`416`	`415`	`namespaceParamName := "namespaces"`
`417`	`416`	`// Handler for standard REST verbs (GET, PUT, POST and DELETE).`
`@@ -467,7 +466,6 @@ func (a *APIInstaller) registerResourceHandlers(path string, storage rest.Storag`
`467`	`466`	`// DEPRECATED in 1.11`
`468`	`467`	`actions = appendIf(actions, action{"WATCHLIST", "watch/" + resource, params, namer, true}, allowWatchList)`
`469`	`468`	`}`
`470`		`- break`
`471`	`469`	`}`
`472`	`470`
`473`	`471`	`// Create Routes for the actions.`