Sarkar Default fix

raphael-s-steiner · raphael-s-steiner · commit 1603fccfe48f · 2025-11-12T08:49:50.000+01:00
diff --git a/include/osp/coarser/Sarkar/Sarkar.hpp b/include/osp/coarser/Sarkar/Sarkar.hpp
@@ -609,9 +609,12 @@ std::vector<std::vector<vertex_idx_t<Graph_t_in>>> Sarkar<Graph_t_in, Graph_t_ou
             }
             break;
 
-        default: // TODO check if it makes sense
+        default:
             {
-                diff = singleContraction(params.commCost, dag_in, expansionMap);
+                #ifdef __cpp_lib_unreachable
+                    std::unreachable();
+                #endif
+                assert(false);
             }
             break;
     }

Original file line number	Diff line number	Diff line change
`@@ -609,9 +609,12 @@ std::vector<std::vector<vertex_idx_t<Graph_t_in>>> Sarkar<Graph_t_in, Graph_t_ou`
`609`	`609`	`}`
`610`	`610`	`break;`
`611`	`611`
`612`		`- default: // TODO check if it makes sense`
	`612`	`+ default:`
`613`	`613`	`{`
`614`		`- diff = singleContraction(params.commCost, dag_in, expansionMap);`
	`614`	`+ #ifdef __cpp_lib_unreachable`
	`615`	`+ std::unreachable();`
	`616`	`+ #endif`
	`617`	`+ assert(false);`
`615`	`618`	`}`
`616`	`619`	`break;`
`617`	`620`	`}`