fixed comments in typedef (#404)

* fixed comments in typedef

Author: AlexHaxe

CHANGES.md

@@ -7,6 +7,8 @@
 - Fixed BkOpen childs in token tree parser [#398](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/398)
 - Fixed bad offset crash with C++ build on7 Windows 10 [#398](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/398)
 - Fixed object declaration handling [#399](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/399)
+- Fixed false positives for files with UTF-8 characters when running as vscode-checkstyle [#402](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/402)
+- Fixed comments in typedefs [#404](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/404)
 - Refactored content handling to use Bytes instead of String (should fix [#98](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/98)) [#402](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/402)
 - Added unittests for ParserQueue and CheckerPool [#393](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/393)
 - Added unittests for TokenTree structure verification [#400](https://github.com/HaxeCheckstyle/haxe-checkstyle/issues/400)

src/checkstyle/token/walk/WalkTypedefBody.hx

@@ -9,6 +9,8 @@ class WalkTypedefBody {
 			while (progress.streamHasChanged()) {
 				switch (stream.token()) {
 					case BrClose: break;
+					case Comment(_), CommentLine(_):
+						WalkComment.walkComment(stream, openTok);
 					default:
 						WalkFieldDef.walkFieldDef(stream, openTok);
 				}

test/token/TokenTreeBuilderParsingTest.hx

@@ -35,6 +35,7 @@ class TokenTreeBuilderParsingTest {
 		assertCodeParses(BLOCK_OBJECT_DECL_SAMPLES_ISSUE_396_2);
 		assertCodeParses(BLOCK_OBJECT_DECL_SAMPLES_ISSUE_396_3);
 		assertCodeParses(BLOCK_OBJECT_DECL_WITH_TERNARY);
+		assertCodeParses(TYPEDEF_COMMENTS);
 	}
 
 	public function assertCodeParses(code:String, ?pos:PosInfos) {
@@ -456,4 +457,15 @@ abstract TokenTreeBuilderParsingTests(String) to String {
 			};
 		}
 	}";
+
+	var TYPEDEF_COMMENTS = "
+	typedef CheckFile = {
+		// °
+		var name:String;
+		// öäü
+		var content:String;
+		// €łµ
+		var index:Int;
+		// æ@ð
+	}";
 }

test/token/verify/VerifyTokenTreeTest.hx

@@ -74,6 +74,38 @@ class VerifyTokenTreeTest {
 		block.childLast().is(BrClose).noChilds();
 	}
 
+	@Test
+	public function testTypedefComments() {
+		var root:IVerifyTokenTree = buildTokenTree(TokenTreeBuilderParsingTests.TYPEDEF_COMMENTS);
+
+		// typedef CheckFile
+		var type:IVerifyTokenTree = root.childFirst().is(Kwd(KwdTypedef)).oneChild().childFirst().is(Const(CIdent("CheckFile")));
+		var brOpen:IVerifyTokenTree = type.childFirst().is(Binop(OpAssign)).oneChild().childFirst().is(BrOpen).childCount(8);
+		brOpen.childAt(0).is(CommentLine(" °"));
+
+		// var name:String;
+		var v:IVerifyTokenTree = brOpen.childAt(1).is(Kwd(KwdVar)).oneChild().childFirst().is(Const(CIdent("name"))).childCount(2);
+		v.childFirst().is(DblDot).oneChild().childFirst().is(Const(CIdent("String"))).noChilds();
+		v.childLast().is(Semicolon).noChilds();
+
+		brOpen.childAt(2).is(CommentLine(" öäü")).noChilds();
+
+		// var content:String;
+		v = brOpen.childAt(3).is(Kwd(KwdVar)).oneChild().childFirst().is(Const(CIdent("content"))).childCount(2);
+		v.childFirst().is(DblDot).oneChild().childFirst().is(Const(CIdent("String"))).noChilds();
+		v.childLast().is(Semicolon).noChilds();
+
+		brOpen.childAt(4).is(CommentLine(" €łµ")).noChilds();
+
+		// var index:Int;
+		v = brOpen.childAt(5).is(Kwd(KwdVar)).oneChild().childFirst().is(Const(CIdent("index"))).childCount(2);
+		v.childFirst().is(DblDot).oneChild().childFirst().is(Const(CIdent("Int"))).noChilds();
+		v.childLast().is(Semicolon).noChilds();
+
+		brOpen.childAt(6).is(CommentLine(" æ@ð")).noChilds();
+		brOpen.childLast().is(BrClose).noChilds();
+	}
+
 	function buildTokenTree(content:String):IVerifyTokenTree {
 		var builder:TestTokenTreeBuilder = new TestTokenTreeBuilder(content);
 		var root:TokenTree = new TokenTree(null, null, -1);