feat(compiler): allow trailing comma in array literal

closes angular#20773
trotyl · Feb 17, 2018 · c5f26a2 · c5f26a2
1 parent f693be3
commit c5f26a2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/compiler/src/expression_parser/parser.ts b/packages/compiler/src/expression_parser/parser.ts
@@ -595,11 +595,15 @@ export class _ParseAST {
 
   parseExpressionList(terminator: number): AST[] {
     const result: AST[] = [];
-    if (!this.next.isCharacter(terminator)) {
-      do {
+
+    do {
+      if (!this.next.isCharacter(terminator)) {
         result.push(this.parsePipe());
-      } while (this.optionalCharacter(chars.$COMMA));
-    }
+      } else {
+        break;
+      }
+    } while (this.optionalCharacter(chars.$COMMA));
+
     return result;
   }
 

diff --git a/packages/compiler/test/expression_parser/parser_spec.ts b/packages/compiler/test/expression_parser/parser_spec.ts
@@ -163,6 +163,7 @@ import {validate} from './validator';
           checkAction('[]');
           checkAction('[].length');
           checkAction('[1, 2].length');
+          checkAction('[1, 2,]', '[1, 2]');
         });
 
         it('should parse map', () => {