From cfbbdf7bf01d7649e9ac8938c1ca4790e04a4413 Mon Sep 17 00:00:00 2001 From: Matthias Radestock Date: Mon, 3 Jul 2017 03:19:52 +0100 Subject: [PATCH] refactor: optimise report.Sets.Delete() going empty Mainly for consistencty; we do something similar NodeSet.Delete(). --- report/sets.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/report/sets.go b/report/sets.go index d73d5d7bf..61d7ec71f 100644 --- a/report/sets.go +++ b/report/sets.go @@ -47,9 +47,11 @@ func (s Sets) Delete(key string) Sets { if s.psMap == nil { return emptySets } - return Sets{ - psMap: s.psMap.Delete(key), + psMap := s.psMap.Delete(key) + if psMap.IsNil() { + return emptySets } + return Sets{psMap: psMap} } // Lookup returns the sets stored under key.