commit 443039a58f35a84bde67eed29e611fb6546cd137
author Ben Clayton <bclayton@google.com> Wed Nov 04 15:48:20 2020 +0000
committer Commit Bot service account <commit-bot@chromium.org> Wed Nov 04 15:48:20 2020 +0000
tree cadbf6260ca50d9dbd07b597a0e06d44db078313
parent 67c37197d6f60313a07a0610044d5ee410b80c76

wsgl parser: Use expect() for call r-parenthesis

Keeps error message consistent. Reduces code.

Bug: tint:282
Change-Id: Ibbbb98bb6f080dd338af7de415c6e86298d910e3
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/31730
Commit-Queue: Ben Clayton <bclayton@google.com>
Commit-Queue: dan sinclair <dsinclair@chromium.org>
Reviewed-by: dan sinclair <dsinclair@chromium.org>

src/reader/wgsl/parser_impl.cc

diff --git a/src/reader/wgsl/parser_impl.cc b/src/reader/wgsl/parser_impl.cc
index 2a63c4a..0176c83 100644
--- a/src/reader/wgsl/parser_impl.cc
+++ b/src/reader/wgsl/parser_impl.cc
@@ -2587,11 +2587,8 @@
       return nullptr;
   }
 
-  t = next();
-  if (!t.IsParenRight()) {
-    add_error(t, "missing ) for call statement");
+  if (!expect("call statement", Token::Type::kParenRight))
     return nullptr;
-  }
 
   return std::make_unique<ast::CallStatement>(
       std::make_unique<ast::CallExpression>(
@@ -2763,11 +2760,9 @@
         return nullptr;
     }
 
-    t = next();
-    if (!t.IsParenRight()) {
-      add_error(t, "missing ) for call expression");
+    if (!expect("call expression", Token::Type::kParenRight))
       return nullptr;
-    }
+
     expr = std::make_unique<ast::CallExpression>(source, std::move(prefix),
                                                  std::move(params));
   } else if (t.IsPeriod()) {

src/reader/wgsl/parser_impl_call_stmt_test.cc

diff --git a/src/reader/wgsl/parser_impl_call_stmt_test.cc b/src/reader/wgsl/parser_impl_call_stmt_test.cc
index 9f567e9..2ab8b19 100644
--- a/src/reader/wgsl/parser_impl_call_stmt_test.cc
+++ b/src/reader/wgsl/parser_impl_call_stmt_test.cc
@@ -63,7 +63,7 @@
   auto* p = parser("a(");
   auto e = p->statement();
   ASSERT_TRUE(p->has_error());
-  EXPECT_EQ(p->error(), "1:3: missing ) for call statement");
+  EXPECT_EQ(p->error(), "1:3: expected ')' for call statement");
 }
 
 TEST_F(ParserImplTest, Statement_Call_Missing_Semi) {
@@ -77,7 +77,7 @@
   auto* p = parser("a(b c);");
   auto e = p->statement();
   ASSERT_TRUE(p->has_error());
-  EXPECT_EQ(p->error(), "1:5: missing ) for call statement");
+  EXPECT_EQ(p->error(), "1:5: expected ')' for call statement");
 }
 
 }  // namespace

src/reader/wgsl/parser_impl_error_msg_test.cc

diff --git a/src/reader/wgsl/parser_impl_error_msg_test.cc b/src/reader/wgsl/parser_impl_error_msg_test.cc
index 73b61d7..e035f9f 100644
--- a/src/reader/wgsl/parser_impl_error_msg_test.cc
+++ b/src/reader/wgsl/parser_impl_error_msg_test.cc
@@ -119,14 +119,14 @@
 
 TEST_F(ParserImplErrorTest, CallExprMissingRParen) {
   EXPECT("fn f() -> void { x = f(1.; }",
-         "test.wgsl:1:26 error: missing ) for call expression\n"
+         "test.wgsl:1:26 error: expected ')' for call expression\n"
          "fn f() -> void { x = f(1.; }\n"
          "                         ^\n");
 }
 
 TEST_F(ParserImplErrorTest, CallStmtMissingRParen) {
   EXPECT("fn f() -> void { f(1.; }",
-         "test.wgsl:1:22 error: missing ) for call statement\n"
+         "test.wgsl:1:22 error: expected ')' for call statement\n"
          "fn f() -> void { f(1.; }\n"
          "                     ^\n");
 }

src/reader/wgsl/parser_impl_postfix_expression_test.cc

diff --git a/src/reader/wgsl/parser_impl_postfix_expression_test.cc b/src/reader/wgsl/parser_impl_postfix_expression_test.cc
index 4ec04b2..ab16e53 100644
--- a/src/reader/wgsl/parser_impl_postfix_expression_test.cc
+++ b/src/reader/wgsl/parser_impl_postfix_expression_test.cc
@@ -144,7 +144,7 @@
   auto e = p->postfix_expression();
   ASSERT_TRUE(p->has_error());
   ASSERT_EQ(e, nullptr);
-  EXPECT_EQ(p->error(), "1:3: missing ) for call expression");
+  EXPECT_EQ(p->error(), "1:3: expected ')' for call expression");
 }
 
 TEST_F(ParserImplTest, PostfixExpression_MemberAccessor) {