Skip to content

Commit 06a8e9d

Browse files
x1z53colinkiama
andauthored
SpaceBeforeParenCheck: fix column offset (#196)
* SpaceBeforeParenCheck: fix column offset Closes: #195 * Fix SpaceBeforeParens check offsets in tests --------- Co-authored-by: Colin Kiama <colinkiama@gmail.com>
1 parent 9212aa4 commit 06a8e9d

3 files changed

Lines changed: 9 additions & 9 deletions

File tree

‎lib/Checks/SpaceBeforeParenCheck.vala‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public class ValaLint.Checks.SpaceBeforeParenCheck : Check {
3232
foreach (ParseResult r in parse_result) {
3333
if (r.type == ParseType.DEFAULT) {
3434
add_regex_mistake ("""[^_\s{\[\(\)!~]\(""", _("Expected space before paren"), r,
35-
ref mistake_list, 1, 1);
35+
ref mistake_list, 1, 0);
3636
}
3737
}
3838
}

‎test/ContentTest.vala‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ class ContentTest : GLib.Object {
5959
var general_warnings = CheckTest.FileTestMistakeList ();
6060
general_warnings.add ("using-directive", 3);
6161
general_warnings.add ("naming-convention", 3);
62-
general_warnings.add ("space-before-paren", 2, 26, 27);
62+
general_warnings.add ("space-before-paren", 2, 25, 26);
6363
general_warnings.add ("note", 1, 21, 25, "TODO");
6464
general_warnings.add ("note", 1, 21, 38, "TODO: Lorem ipsum");
6565
general_warnings.add ("double-spaces", 2, 25, 27);
@@ -105,9 +105,9 @@ class ContentTest : GLib.Object {
105105
check_contents_for_mistake (CheckTest.get_test_file ("note-check.vala"), note_warnings);
106106

107107
var space_before_paren_warnings = CheckTest.FileTestMistakeList ();
108-
space_before_paren_warnings.add ("space-before-paren", 4, 9, 10);
109-
space_before_paren_warnings.add ("no-space", 5, 14, 15);
110-
space_before_paren_warnings.add ("space-before-paren", 0, 14, 15);
108+
space_before_paren_warnings.add ("space-before-paren", 4, 8, 9);
109+
space_before_paren_warnings.add ("space-before-paren", 5, 13, 14);
110+
space_before_paren_warnings.add ("no-space", 0, 14, 15);
111111
space_before_paren_warnings.add ("no-space", 0, 17, 18);
112112
space_before_paren_warnings.add ("no-space", 0, 18, 19);
113113
check_contents_for_mistake (CheckTest.get_test_file ("space-before-paren-check.vala"),

‎test/FileTest.vala‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ class FileTest : GLib.Object {
5454
var general_warnings = CheckTest.FileTestMistakeList ();
5555
general_warnings.add ("using-directive", 3);
5656
general_warnings.add ("naming-convention", 3);
57-
general_warnings.add ("space-before-paren", 2, 26, 27);
57+
general_warnings.add ("space-before-paren", 2, 25, 26);
5858
general_warnings.add ("note", 1, 21, 25, "TODO");
5959
general_warnings.add ("note", 1, 21, 38, "TODO: Lorem ipsum");
6060
general_warnings.add ("double-spaces", 2, 25, 27);
@@ -100,9 +100,9 @@ class FileTest : GLib.Object {
100100
check_file_for_mistake (CheckTest.get_test_file ("note-check.vala"), note_warnings);
101101

102102
var space_before_paren_warnings = CheckTest.FileTestMistakeList ();
103-
space_before_paren_warnings.add ("space-before-paren", 4, 9, 10);
104-
space_before_paren_warnings.add ("no-space", 5, 14, 15);
105-
space_before_paren_warnings.add ("space-before-paren", 0, 14, 15);
103+
space_before_paren_warnings.add ("space-before-paren", 4, 8, 9);
104+
space_before_paren_warnings.add ("space-before-paren", 5, 13, 14);
105+
space_before_paren_warnings.add ("no-space", 0, 14, 15);
106106
space_before_paren_warnings.add ("no-space", 0, 17, 18);
107107
space_before_paren_warnings.add ("no-space", 0, 18, 19);
108108
check_file_for_mistake (CheckTest.get_test_file ("space-before-paren-check.vala"),

0 commit comments

Comments
 (0)