Don't parse NamedTuple type without comma seperator (#5981)

makenowjust · RX14 · commit 2f2c04faa4ce · 2018-04-21T16:49:17.000+01:00
Fixed #5979
diff --git a/spec/compiler/parser/parser_spec.cr b/spec/compiler/parser/parser_spec.cr
@@ -500,6 +500,7 @@ describe "Parser" do
   it_parses "Foo(x: U)", Generic.new("Foo".path, [] of ASTNode, named_args: [NamedArgument.new("x", "U".path)])
   it_parses "Foo(x: U, y: V)", Generic.new("Foo".path, [] of ASTNode, named_args: [NamedArgument.new("x", "U".path), NamedArgument.new("y", "V".path)])
   assert_syntax_error "Foo(T, x: U)"
+  assert_syntax_error "Foo(x: T y: U)"
 
   it_parses %(Foo("foo bar": U)), Generic.new("Foo".path, [] of ASTNode, named_args: [NamedArgument.new("foo bar", "U".path)])
   it_parses %(Foo("foo": U, "bar": V)), Generic.new("Foo".path, [] of ASTNode, named_args: [NamedArgument.new("foo", "U".path), NamedArgument.new("bar", "V".path)])
diff --git a/src/compiler/crystal/syntax/parser.cr b/src/compiler/crystal/syntax/parser.cr
@@ -4466,9 +4466,8 @@ module Crystal
         skip_space_or_newline
 
         named_args << NamedArgument.new(name, type)
-        if @token.type == :","
-          next_token_skip_space_or_newline
-        end
+        break unless @token.type == :","
+        next_token_skip_space_or_newline
       end
 
       named_args
