parser: Allow empty member-initializer and accept trailing comma

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 03b221b9cbd28264472399d6fab8e435e99d3f42..79656a8161dd530ab47e8e842a7d8550728a5f5e 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -823,6 +823,9 @@ TESTS = \
        parser/lock-statement.vala \
        parser/main-block.vala \
        parser/member-access-pointer.vala \
+       parser/member-initializer.vala \
+       parser/member-initializer-empty.vala \
+       parser/member-initializer-trailing-comma.vala \
        parser/method-no-abstract-override.test \
        parser/method-no-abstract-virtual-override.test \
        parser/method-no-abstract-virtual.test \

diff --git a/tests/parser/member-initializer-empty.vala b/tests/parser/member-initializer-empty.vala
new file mode 100644 (file)
index 0000000..732503b
--- /dev/null
+++ b/tests/parser/member-initializer-empty.vala
@@ -0,0 +1,7 @@
+class Foo {
+       public int i;
+}
+
+void main () {
+       var foo = new Foo () {};
+}

diff --git a/tests/parser/member-initializer-trailing-comma.vala b/tests/parser/member-initializer-trailing-comma.vala
new file mode 100644 (file)
index 0000000..8e7503c
--- /dev/null
+++ b/tests/parser/member-initializer-trailing-comma.vala
@@ -0,0 +1,9 @@
+class Foo {
+       public int i;
+}
+
+void main () {
+       var foo = new Foo () {
+               i = 42,
+       };
+}

diff --git a/tests/parser/member-initializer.vala b/tests/parser/member-initializer.vala
new file mode 100644 (file)
index 0000000..7795c14
--- /dev/null
+++ b/tests/parser/member-initializer.vala
@@ -0,0 +1,9 @@
+class Foo {
+       public int i;
+}
+
+void main () {
+       var foo = new Foo () {
+               i = 23
+       };
+}

diff --git a/vala/valaparser.vala b/vala/valaparser.vala
index d3f0875feafa552ac898b89359bad3ed4929b3cb..41cd47698dc5ea155c7ca30f09a7e357aa13dec9 100644 (file)
--- a/vala/valaparser.vala
+++ b/vala/valaparser.vala
@@ -1052,9 +1052,13 @@ public class Vala.Parser : CodeVisitor {
        List<MemberInitializer> parse_object_initializer () throws ParseError {
                var list = new ArrayList<MemberInitializer> ();
                if (accept (TokenType.OPEN_BRACE)) {
-                       do {
+                       while (current () != TokenType.CLOSE_BRACE) {
                                list.add (parse_member_initializer ());
-                       } while (accept (TokenType.COMMA));
+
+                               if (!accept (TokenType.COMMA)) {
+                                       break;
+                               }
+                       }
                        expect (TokenType.CLOSE_BRACE);
                }
                return list;