From e8215cc57798b9d057713f2b28e0bfd0ef72b70f Mon Sep 17 00:00:00 2001 From: Gyu-Ho Lee Date: Wed, 25 Nov 2015 09:41:27 -0800 Subject: [PATCH] etcdctl: remove duplicate exit line `handleError` already exits with the exit code in arguments. `os.Exit(1)` is never executed in this case. --- etcdctl/command/util.go | 1 - 1 file changed, 1 deletion(-) diff --git a/etcdctl/command/util.go b/etcdctl/command/util.go index 99975804d..9b3452338 100644 --- a/etcdctl/command/util.go +++ b/etcdctl/command/util.go @@ -210,7 +210,6 @@ func mustNewClient(c *cli.Context) client.Client { fmt.Fprintf(os.Stderr, "Try '--no-sync' if you want to access non-published client endpoints(%s).\n", strings.Join(hc.Endpoints(), ",")) } handleError(ExitServerError, err) - os.Exit(1) } if debug { fmt.Fprintf(os.Stderr, "got endpoints(%s) after sync\n", strings.Join(hc.Endpoints(), ","))