Merge pull request #213 from JuliaIO/td-trailing-semicolons

Allow trailing semicolon after message definition

Author: Drvi

src/parsing/proto_types.jl

@@ -369,6 +369,7 @@ function _parse_message_body(ps::ParserState, name, definitions, name_prefix)
             push!(fields, group)
             definitions[group.type.name] = group.type
         elseif accept(ps, Tokens.RBRACE)
+            accept(ps, Tokens.SEMICOLON)
             break
         else
             push!(fields, parse_field(ps))

test/test_parser.jl

@@ -72,6 +72,15 @@ end
         @test p.definitions["A"].element_values == [0]
     end
 
+    @testset "Trailing semicolon is fine" begin
+        s, p, ctx = translate_simple_proto("message A {}; enum B { b = 0; };")
+
+        @test haskey(p.definitions, "A")
+        @test p.definitions["A"] isa Parsers.MessageType
+        @test haskey(p.definitions, "B")
+        @test p.definitions["B"] isa Parsers.EnumType
+    end
+
     @testset "Single enum with `allow_alias = true` proto file" begin
         s, p, ctx = translate_simple_proto("enum A { option allow_alias = true; a = 0; b = 0; }")