Merge pull request #36 from etilite/fix-check-capital

fix: panic due incorrect issue's position column resolution in checkCapital

Author: tetafro

checks.go

@@ -171,7 +171,7 @@ func checkCapital(c comment) []Issue {
 		if state == endOfSentence && unicode.IsLower(r) {
 			pp = append(pp, position{
 				line:   pos.line,
-				column: runeToByteColumn(c.text, pos.column),
+				column: runeToByteColumn(c.lines[pos.line-1], pos.column),
 			})
 		}
 		state = empty

checks_test.go

@@ -337,6 +337,17 @@ func TestCheckCapital(t *testing.T) {
 				{Pos: token.Position{Line: 1, Column: 23}},
 			},
 		},
+		{
+			name: "issue position column resolved from correct line",
+			comment: comment{
+				lines: []string{"// Кириллица.", "// Issue. here."},
+				text:  " Кириллица.\n Issue. here.",
+				start: start,
+			},
+			issues: []Issue{
+				{Pos: token.Position{Line: 2, Column: 11}},
+			},
+		},
 		{
 			name: "sentence with leading spaces",
 			comment: comment{