Be idiomatic. Use a switch instead of if/else

This commit is contained in:
Sean Chittenden 2016-04-29 11:35:33 -07:00
parent 01fac00f20
commit 1df788a73c

View File

@ -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
}
}