Include location in error message

Author: xitep

src/parser/mod.rs

@@ -17229,20 +17229,30 @@ impl<'a> Parser<'a> {
 
             self.expect_keyword_is(Keyword::THEN)?;
 
+            macro_rules! not_allowed {
+                ($action:literal, $not_allowed_in_clauses:pat) => {
+                    if matches!(clause_kind, $not_allowed_in_clauses) {
+                        return parser_err!(
+                            format_args!(
+                                concat!($action, " is not allowed in a {} merge clause"),
+                                clause_kind
+                            ),
+                            self.get_current_token().span.start
+                        );
+                    }
+                };
+            }
             let merge_clause = match self.parse_one_of_keywords(&[
                 Keyword::UPDATE,
                 Keyword::INSERT,
                 Keyword::DELETE,
             ]) {
                 Some(Keyword::UPDATE) => {
-                    if matches!(
-                        clause_kind,
+                    not_allowed!(
+                        "UPDATE",
                         MergeClauseKind::NotMatched | MergeClauseKind::NotMatchedByTarget
-                    ) {
-                        return Err(ParserError::ParserError(format!(
-                            "UPDATE is not allowed in a {clause_kind} merge clause"
-                        )));
-                    }
+                    );
+
                     let update_token = self.get_current_token().clone();
                     self.expect_keyword_is(Keyword::SET)?;
                     MergeAction::Update {
@@ -17251,28 +17261,22 @@ impl<'a> Parser<'a> {
                     }
                 }
                 Some(Keyword::DELETE) => {
-                    if matches!(
-                        clause_kind,
+                    not_allowed!(
+                        "DELETE",
                         MergeClauseKind::NotMatched | MergeClauseKind::NotMatchedByTarget
-                    ) {
-                        return Err(ParserError::ParserError(format!(
-                            "DELETE is not allowed in a {clause_kind} merge clause"
-                        )));
-                    }
+                    );
+
                     let delete_token = self.get_current_token().clone();
                     MergeAction::Delete {
                         delete_token: delete_token.into(),
                     }
                 }
                 Some(Keyword::INSERT) => {
-                    if !matches!(
-                        clause_kind,
-                        MergeClauseKind::NotMatched | MergeClauseKind::NotMatchedByTarget
-                    ) {
-                        return Err(ParserError::ParserError(format!(
-                            "INSERT is not allowed in a {clause_kind} merge clause"
-                        )));
-                    }
+                    not_allowed!(
+                        "INSERT",
+                        MergeClauseKind::Matched | MergeClauseKind::NotMatchedBySource
+                    );
+
                     let insert_token = self.get_current_token().clone();
                     let is_mysql = dialect_of!(self is MySqlDialect);
 
@@ -17295,9 +17299,10 @@ impl<'a> Parser<'a> {
                     })
                 }
                 _ => {
-                    return Err(ParserError::ParserError(
-                        "expected UPDATE, DELETE or INSERT in merge clause".to_string(),
-                    ));
+                    return parser_err!(
+                        "expected UPDATE, DELETE or INSERT in merge clause",
+                        self.peek_token_ref().span.start
+                    );
                 }
             };
             clauses.push(MergeClause {