Merge pull request #7607 from asterite/bug/disallow-space-between-enu…

…m-members

Parser: disallow space between enum members

spec/compiler/parser/parser_spec.cr

@@ -1361,6 +1361,8 @@ module Crystal
 
     it_parses "enum Foo; @[Bar]; end", EnumDef.new("Foo".path, [Annotation.new("Bar".path)] of ASTNode)
 
+    assert_syntax_error "enum Foo; A B; end", "expecting ',', ';', 'end' or newline after enum member"
+
     it_parses "1.[](2)", Call.new(1.int32, "[]", 2.int32)
     it_parses "1.[]?(2)", Call.new(1.int32, "[]?", 2.int32)
     it_parses "1.[]=(2, 3)", Call.new(1.int32, "[]=", 2.int32, 3.int32)

src/compiler/crystal/syntax/parser.cr

@@ -5522,17 +5522,20 @@ module Crystal
           next_token_skip_space
           if @token.type == :"="
             next_token_skip_space_or_newline
-
             constant_value = parse_logical_or
-            next_token_skip_statement_end
           else
             constant_value = nil
-            skip_statement_end
           end
 
+          skip_space
+
           case @token.type
-          when :",", :";"
+          when :",", :";", :NEWLINE, :EOF
             next_token_skip_statement_end
+          else
+            unless @token.keyword?(:end)
+              raise "expecting ',', ';', 'end' or newline after enum member", location
+            end
           end
 
           arg = Arg.new(constant_name, constant_value).at(location).at_end(constant_value || location)