Clean-up core signals handler by removing unnecessary cruft

Justin Reagor · Justin Reagor · commit 87bcd66bbc0e · 2017-12-07T09:34:32.000-05:00
diff --git a/core/signals.go b/core/signals.go
@@ -19,17 +19,12 @@ func (a *App) handleSignals() {
 		for {
 			sig := <-recvSig
 			switch sig {
-			case syscall.SIGINT:
+			case syscall.SIGINT, syscall.SIGTERM:
 				a.Terminate()
-				return
-			case syscall.SIGTERM:
-				a.Terminate()
-				return
 			case syscall.SIGHUP, syscall.SIGUSR2:
 				if s := toString(sig); s != "" {
 					a.SignalEvent(s)
 				}
-			default:
 			}
 		}
 	}()

Original file line number	Diff line number	Diff line change
`@@ -19,17 +19,12 @@ func (a *App) handleSignals() {`
`19`	`19`	`for {`
`20`	`20`	`sig := <-recvSig`
`21`	`21`	`switch sig {`
`22`		`- case syscall.SIGINT:`
	`22`	`+ case syscall.SIGINT, syscall.SIGTERM:`
`23`	`23`	`a.Terminate()`
`24`		`- return`
`25`		`- case syscall.SIGTERM:`
`26`		`- a.Terminate()`
`27`		`- return`
`28`	`24`	`case syscall.SIGHUP, syscall.SIGUSR2:`
`29`	`25`	`if s := toString(sig); s != "" {`
`30`	`26`	`a.SignalEvent(s)`
`31`	`27`	`}`
`32`		`- default:`
`33`	`28`	`}`
`34`	`29`	`}`
`35`	`30`	`}()`