remove unnecessary TODOs

Author: larsgottesbueren

algorithm/hla_flowcutter.h

@@ -48,9 +48,6 @@ namespace hyper {
 		flow_t cutsize = 0;
 
 		std::vector<nodeid> flow_from, flow_to;
-		//Bitvector has_flow;	//TODO can be eliminated by using flow_from != invalid_node
-
-
 
 		search_space source_nodes, target_nodes;						//could be turned into one since reachable sets must be disjoint, unless the node got pierced. need separate timestamps for source_assimilated, target_assimilated
 		hyperedge_search_space source_hyperedges, target_hyperedges;	//could be turned into one. odd timestamps for source, even for target

algorithm/multicutter_disconnected.h

@@ -449,7 +449,7 @@ class DisconnectedMultiCutter {
 			includeNextCCWithFlowCutterBasicVariantCacheFriendlyWithoutIndex(front, csize, ccsizes_sofar, global_perf_balance, ngmc, gmc);
 		}
 
-		{				//special case for including the entire component with zero cut. TODO recall why j=0 was ignored
+		{
 			for (nodeid i = 0; i <= std::min(ccsizes_sofar + csize, global_perf_balance) && csize + i <= global_perf_balance; i++)
 				if (ngmc[i] != MAX_FLOW)
 					gmc[csize + i] = ngmc[i];

definitions.h

@@ -44,15 +44,6 @@ namespace hyper {
 	using partitionid = uint32_t;
 	using flow_t = int32_t;
 
-	//TODO substitute usages of nodeid and netid by Node and Hyperedge so the compiler catches more mistakes. also do NodeIndexable vectors
-	struct Node {
-		nodeid value;
-	};
-
-	struct Hyperedge {
-		netid value;
-	};
-
 	static constexpr arcid invalid_arc = std::numeric_limits<arcid>::max();
 	static constexpr nodeid invalid_node = std::numeric_limits<nodeid>::max();
 	static constexpr netid invalid_hyperedge = std::numeric_limits<netid>::max();