From c553b6b950870b299b40e0153818430c649b76cf Mon Sep 17 00:00:00 2001 From: Wouter van Oortmerssen Date: Thu, 21 Aug 2014 15:17:45 -0700 Subject: [PATCH] The list of enum values is now allowed to end in a comma. Bug: 16490424 Change-Id: Ic3dd5f06efb5cf2dc4aefbd3f2db64c7b59b6b93 Tested: on OS X. --- src/idl_parser.cpp | 2 +- tests/monster_test.fbs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/idl_parser.cpp b/src/idl_parser.cpp index 8b7f78e..bf550df 100644 --- a/src/idl_parser.cpp +++ b/src/idl_parser.cpp @@ -725,7 +725,7 @@ void Parser::ParseEnum(bool is_union) { if (prevsize && enum_def.vals.vec[prevsize - 1]->value >= ev.value) Error("enum values must be specified in ascending order"); } - } while (IsNext(',')); + } while (IsNext(',') && token_ != '}'); Expect('}'); if (enum_def.attributes.Lookup("bit_flags")) { for (auto it = enum_def.vals.vec.begin(); it != enum_def.vals.vec.end(); diff --git a/tests/monster_test.fbs b/tests/monster_test.fbs index 7494144..9f3920d 100755 --- a/tests/monster_test.fbs +++ b/tests/monster_test.fbs @@ -4,7 +4,7 @@ include "include_test1.fbs"; namespace MyGame.Example; -enum Color:byte (bit_flags) { Red = 0, Green, Blue = 3 } +enum Color:byte (bit_flags) { Red = 0, Green, Blue = 3, } union Any { Monster } // TODO: add more elements -- 2.7.4