mirrors/Odin
1
0
Fork 0
mirror of https://github.com/odin-lang/Odin.git synced 2026-04-28 09:13:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix duplicate suggestions and add missing newline

Browse Source
This commit is contained in:
Laytan
2024-05-09 19:21:39 +02:00
parent 8a1e7bb6fb
commit 98827c867d
2 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/check_expr.cpp
View File

@@ -6201,7 +6201,7 @@ gb_internal bool evaluate_where_clauses(CheckerContext *ctx, Ast *call_expr, Sco
error(c, "Prefer to separate 'where' clauses with a comma rather than '&&'");
gbString x = expr_to_string(c->BinaryExpr.left);
gbString y = expr_to_string(c->BinaryExpr.right);
error_line("\tSuggestion: '%s, %s'", x, y);
error_line("\tSuggestion: '%s, %s'\n", x, y);
gb_string_free(y);
gb_string_free(x);
}

8
src/error.cpp
View File

@@ -719,9 +719,13 @@ gb_internal void print_all_errors(void) {
}
}
if (it.str.len-it.pos > 0) {
array_add_elems(&prev_ev->msg, it.str.text+it.pos, it.str.len-it.pos);
// Merge additional text (suggestions for example) into the previous error.
String current = {prev_ev->msg.data, prev_ev->msg.count};
String addition = {it.str.text+it.pos, it.str.len-it.pos};
if (addition.len > 0 && !string_contains_string(current, addition)) {
array_add_elems(&prev_ev->msg, addition.text, addition.len);
}
array_free(&ev.msg);
array_ordered_remove(&global_error_collector.error_values, i);
} else {