Skip to content

Commit 8315558

Browse files
committed
Test: delete FIXME messages in CFL_reach algorithm
1 parent e8c2417 commit 8315558

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

experimental/algorithm/LAGraph_CFL_reachability.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@
1515
// Querying Using Linear Algebra", URL:
1616
// https://disser.spbu.ru/files/2022/disser_azimov.pdf
1717

18-
// FIXME: some of the code below is not covered by the test suite, "make cov".
19-
// See the FIXMEs below.
2018

2119
#define LG_FREE_WORK \
2220
{ \
@@ -175,7 +173,6 @@ GrB_Info LAGraph_CFL_reachability
175173
ADD_TO_MSG("Adjacency matrices with these indexes are null: ");
176174
ADD_TO_MSG("%d", i);
177175
} else {
178-
// FIXME: this case is not tested.
179176
ADD_TO_MSG(", %d", i);
180177
}
181178

@@ -224,7 +221,6 @@ GrB_Info LAGraph_CFL_reachability
224221

225222
// [Variable -> eps]
226223
if (is_rule_eps) {
227-
// FIXME: this case is not tested.
228224
eps_rules[eps_rules_count++] = i;
229225

230226
continue;
@@ -247,7 +243,6 @@ GrB_Info LAGraph_CFL_reachability
247243

248244
if (rule.prod_A < -1 || rule.prod_A >= nonterms_count || rule.prod_B < -1 ||
249245
rule.prod_B >= nonterms_count) {
250-
// FIXME: this case is not tested.
251246
ADD_INDEX_TO_ERROR_RULE(nonterm_err, i);
252247
}
253248

@@ -286,7 +281,6 @@ GrB_Info LAGraph_CFL_reachability
286281
GRB_TRY(GrB_Matrix_nvals(&adj_matrix_nnz, adj_matrices[term_rule.prod_A]));
287282

288283
if (adj_matrix_nnz == 0) {
289-
// FIXME: this case is not tested.
290284
continue;
291285
}
292286

@@ -311,7 +305,6 @@ GrB_Info LAGraph_CFL_reachability
311305

312306
// Rule [Variable -> eps]
313307
for (size_t i = 0; i < eps_rules_count; i++) {
314-
// FIXME: this case is not tested.
315308
LAGraph_rule_WCNF eps_rule = rules[eps_rules[i]];
316309

317310
GxB_eWiseUnion (

0 commit comments

Comments
 (0)