Remove redundant erase call

sebrockm · sebrockm · commit 446dc15cf863 · 2022-07-17T21:23:08.000+02:00
diff --git a/include/boost/graph/transitive_closure.hpp b/include/boost/graph/transitive_closure.hpp
@@ -107,12 +107,11 @@ void transitive_closure(const Graph& g, GraphTC& tc,
             }
         }
         std::sort(adj.begin(), adj.end());
-        typename std::vector< cg_vertex >::iterator di
+        const typename std::vector< cg_vertex >::iterator di
             = std::unique(adj.begin(), adj.end());
-        if (di != adj.end())
-            adj.erase(di, adj.end());
+
         for (typename std::vector< cg_vertex >::const_iterator i = adj.begin();
-             i != adj.end(); ++i)
+             i != di; ++i)
         {
             add_edge(s, *i, CG);
         }

Original file line number	Diff line number	Diff line change
`@@ -107,12 +107,11 @@ void transitive_closure(const Graph& g, GraphTC& tc,`
`107`	`107`	`}`
`108`	`108`	`}`
`109`	`109`	`std::sort(adj.begin(), adj.end());`
`110`		`- typename std::vector< cg_vertex >::iterator di`
	`110`	`+ const typename std::vector< cg_vertex >::iterator di`
`111`	`111`	`= std::unique(adj.begin(), adj.end());`
`112`		`- if (di != adj.end())`
`113`		`- adj.erase(di, adj.end());`
	`112`	`+`
`114`	`113`	`for (typename std::vector< cg_vertex >::const_iterator i = adj.begin();`
`115`		`- i != adj.end(); ++i)`
	`114`	`+ i != di; ++i)`
`116`	`115`	`{`
`117`	`116`	`add_edge(s, *i, CG);`
`118`	`117`	`}`