From 1df788a73cd3273286f44ecaa32de85e9ab4a110 Mon Sep 17 00:00:00 2001 From: Sean Chittenden Date: Fri, 29 Apr 2016 11:35:33 -0700 Subject: [PATCH] Be idiomatic. Use a switch instead of if/else --- physical/consul.go | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/physical/consul.go b/physical/consul.go index f9c26dc732..11dfaf048b 100644 --- a/physical/consul.go +++ b/physical/consul.go @@ -521,10 +521,14 @@ func (c *ConsulBackend) reconcileConsul(registeredServiceID string, activeFunc a tags := serviceTags(active) var reregister bool - if currentVaultService == nil || registeredServiceID == "" { + switch { + case currentVaultService == nil, + registeredServiceID == "": reregister = true - } else { - if len(currentVaultService.Tags) != 1 || currentVaultService.Tags[0] != tags[0] { + default: + switch { + case len(currentVaultService.Tags) != 1, + currentVaultService.Tags[0] != tags[0]: reregister = true } }