Commitdb0dfa0

committed

Unnecessary generator - rewrite as a set comprehension

1 parent87a6441 commitdb0dfa0Copy full SHA for db0dfa0

File tree

-4

lines changed

-4

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ def assert_entries(entries, num_entries, has_conflict=False):`
`211`	`211`	`def_assert_tree_entries(self,entries,num_trees):`
`212`	`212`	`forentryinentries:`
`213`	`213`	`assertlen(entry)==num_trees`
`214`		`-paths=set(e[2]foreinentryife)`
	`214`	`+paths={e[2]foreinentryife}`
`215`	`215`
`216`	`216`	`# only one path per set of entries`
`217`	`217`	`assertlen(paths)==1`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -147,8 +147,8 @@ def _cmp_tree_index(self, tree, index):`
`147`	`147`	`blist.append(blob)`
`148`	`148`	`# END for each blob in tree`
`149`	`149`	`iflen(blist)!=len(index.entries):`
`150`		`-iset=set(k[0]forkinindex.entries.keys())`
`151`		`-bset=set(b.pathforbinblist)`
	`150`	`+iset={k[0]forkinindex.entries.keys()}`
	`151`	`+bset={b.pathforbinblist}`
`152`	`152`	`raiseAssertionError("CMP Failed: Missing entries in index: %s, missing in tree: %s"%`
`153`	`153`	`(bset-iset,iset-bset))`
`154`	`154`	`# END assertion message`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ def test_traverse(self):`
`88`	`88`
`89`	`89`	`# trees and blobs`
`90`	`90`	`assertlen(set(trees)\|set(root.trees))==len(trees)`
`91`		`-assertlen(set(bforbinrootifisinstance(b,Blob))\|set(root.blobs))==len(root.blobs)`
	`91`	`+assertlen({bforbinrootifisinstance(b,Blob)}\|set(root.blobs))==len(root.blobs)`
`92`	`92`	`subitem=trees[0][0]`
`93`	`93`	`assert"/"insubitem.path`
`94`	`94`	`assertsubitem.name==osp.basename(subitem.path)`

Comments

(0)