From: Simon Ruderich <simon@ruderich.org>
Date: Sun, 9 May 2021 10:47:40 +0000 (+0200)
Subject: safcm: use better variable name in hostsToSync()
X-Git-Url: https://ruderich.org/simon/gitweb/?a=commitdiff_plain;h=a8023b74bdd3d85cb0f3a55485887d133d2c0946;p=safcm%2Fsafcm.git

safcm: use better variable name in hostsToSync()

Having nameMatched and hostMatched is confusing. Follow the comment and
use hostAdded which describes the meaning more clearly.
---

diff --git a/cmd/safcm/sync.go b/cmd/safcm/sync.go
index 54df9ec..70aaf47 100644
--- a/cmd/safcm/sync.go
+++ b/cmd/safcm/sync.go
@@ -247,13 +247,13 @@ are only available after the hosts were contacted.
 	nameMatched := make(map[string]bool)
 	// To detect typos we must check all given names but one host can be
 	// matched by multiple names (e.g. two groups with overlapping hosts)
-	hostMatched := make(map[string]bool)
+	hostAdded := make(map[string]bool)
 
 	var res []*config.Host
 	for _, host := range allHosts.List {
 		if nameMap[host.Name] {
 			res = append(res, host)
-			hostMatched[host.Name] = true
+			hostAdded[host.Name] = true
 			nameMatched[host.Name] = true
 		}
 
@@ -264,9 +264,9 @@ are only available after the hosts were contacted.
 		}
 		for _, x := range groups {
 			if nameMap[x] {
-				if !hostMatched[host.Name] {
+				if !hostAdded[host.Name] {
 					res = append(res, host)
-					hostMatched[host.Name] = true
+					hostAdded[host.Name] = true
 				}
 				nameMatched[x] = true
 			}