Skip to content

Commit

Permalink
object_manager.go: Fix internal golint suggestions (#45)
Browse files Browse the repository at this point in the history
Fixes:
===
object_manager.go:45:41: method parameter cloudApiOwned should be cloudAPIOwned
object_manager.go:151:9: should omit 2nd value from range; this loop is equivalent to `for k := range ...`
===
  • Loading branch information
eest authored and johnbelamaric committed Mar 5, 2018
1 parent cb9e30b commit 8d79560
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions object_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ func NewObjectManager(connector IBConnector, cmpType string, tenantID string) *O
return objMgr
}

func (objMgr *ObjectManager) getBasicEA(cloudApiOwned Bool) EA {
func (objMgr *ObjectManager) getBasicEA(cloudAPIOwned Bool) EA {
ea := make(EA)
ea["Cloud API Owned"] = cloudApiOwned
ea["Cloud API Owned"] = cloudAPIOwned
ea["CMP Type"] = objMgr.cmpType
ea["Tenant ID"] = objMgr.tenantID
return ea
Expand Down Expand Up @@ -148,7 +148,7 @@ func (objMgr *ObjectManager) UpdateNetworkViewEA(ref string, addEA EA, removeEA
res.Ea[k] = v
}

for k, _ := range removeEA {
for k := range removeEA {
_, ok := res.Ea[k]
if ok {
delete(res.Ea, k)
Expand Down

0 comments on commit 8d79560

Please sign in to comment.