fix endless loop with case in block (#93)

* fix endless loop with case in block
* prepare release 1.0.9

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## dev branch / next version (1.x.x)
 
+## version 1.0.9 (2018-08-13)
+
+- Fixed endless loop with case and conditionals [#93](https://github.com/HaxeCheckstyle/tokentree/issues/93)
+
 ## version 1.0.8 (2018-08-13)
 
 - Fixed null poiner and detection in `getPOpenType` [#73](https://github.com/HaxeCheckstyle/tokentree/issues/73) + [#88](https://github.com/HaxeCheckstyle/tokentree/issues/88) +  [#91](https://github.com/HaxeCheckstyle/tokentree/issues/91)

haxelib.json

@@ -7,8 +7,8 @@
     "contributors": [
         "AlexHaxe"
     ],
-    "releasenote": "fixed POpen and BrOpen type detection, fixed endless loop with switch and conditionals - see CHANGELOG for details",
-    "version": "1.0.8",
+    "releasenote": "fixed endless loop with switch and conditionals - see CHANGELOG for details",
+    "version": "1.0.9",
     "url": "https://github.com/HaxeCheckstyle/tokentree",
     "dependencies": {}
 }

package.json

@@ -1,6 +1,6 @@
 {
     "name": "tokentree",
-    "version": "1.0.8",
+    "version": "1.0.9",
     "description": "TokenTree library used by haxe-checkstyle",
     "repository": {
         "type": "git",

src/tokentree/walk/WalkBlock.hx

@@ -41,6 +41,8 @@ class WalkBlock {
 				case BkClose, PClose:
 					var child:TokenTree = stream.consumeToken();
 					parent.addChild(child);
+				case Kwd(KwdCase), Kwd(KwdDefault):
+					WalkSwitch.walkSwitchCases(stream, parent);
 				default:
 					WalkStatement.walkStatement(stream, parent);
 			}

src/tokentree/walk/WalkSwitch.hx

@@ -28,10 +28,17 @@ class WalkSwitch {
 		WalkComment.walkComment(stream, switchTok);
 		WalkStatement.walkStatement(stream, switchTok);
 		WalkComment.walkComment(stream, switchTok);
-		var brOpen:TokenTree = stream.consumeTokenDef(BrOpen);
-		switchTok.addChild(brOpen);
-		WalkSwitch.walkSwitchCases(stream, brOpen);
-		brOpen.addChild(stream.consumeTokenDef(BrClose));
+		switch (stream.token()) {
+			case Sharp(_):
+				WalkSharp.walkSharp(stream, parent, WalkSwitch.walkSwitchCases);
+			default:
+		}
+		if (stream.is(BrOpen)) {
+			var brOpen:TokenTree = stream.consumeTokenDef(BrOpen);
+			switchTok.addChild(brOpen);
+			WalkSwitch.walkSwitchCases(stream, brOpen);
+			brOpen.addChild(stream.consumeTokenDef(BrClose));
+		}
 	}
 
 	public static function walkSwitchCases(stream:TokenStream, parent:TokenTree) {

test/tokentree/TokenTreeBuilderParsingTest.hx

@@ -866,6 +866,25 @@ abstract TokenTreeBuilderParsingTests(String) to String {
 					Sys.println (' succedded');
 			}
 		}
+		public static function main() {
+			#if neko
+				switch (Sys.command ('neko', ['tests.n', name, field.name]))
+			#elseif cpp
+				switch (Sys.command ('./bin/FunctionalTest', [name, field.name]))
+			#end {
+				case 0:
+					passed++;
+					Sys.println (' succedded');
+			#if neko
+				case 1:
+					Sys.println(' failed');
+			#end
+				case 1:
+					Sys.println(' failed');
+				case _:
+					Sys.println(' errored');
+			}
+		}
 	}
 	";
 }