From 2d7adb94b82993d46e337683bccf887d1952dcd6 Mon Sep 17 00:00:00 2001 From: Heinz Knutzen Date: Wed, 28 Feb 2024 11:31:54 +0100 Subject: [PATCH] Minor simplification --- go/pkg/pass1/check-service-owner.go | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/go/pkg/pass1/check-service-owner.go b/go/pkg/pass1/check-service-owner.go index 9021d010..a64a9f81 100644 --- a/go/pkg/pass1/check-service-owner.go +++ b/go/pkg/pass1/check-service-owner.go @@ -282,7 +282,7 @@ func (c *spoc) checkServiceOwner(sRules *serviceRules) { // Check if attribute 'multi_owner' could be avoided, // if objects of user and objects of rules are swapped. var userOwner *owner - simpleUser := true + USERS: for _, users := range info.users { for _, user := range users { var o *owner @@ -290,18 +290,18 @@ func (c *spoc) checkServiceOwner(sRules *serviceRules) { o = obj.getOwner() } if o == nil { - simpleUser = false - break + userOwner = nil + break USERS } if userOwner == nil { userOwner = o } else if userOwner != o { - simpleUser = false - break + userOwner = nil + break USERS } } } - if simpleUser && userOwner != nil { + if userOwner != nil { c.warn("Unnecessary 'multi_owner' at %s\n"+ " All 'user' objects belong to single %s.\n"+ " Either swap objects of 'user' and objects of rules,\n"+