Fixes based on comments

Author: romanoff

src/dialect/snowflake.rs

@@ -250,11 +250,10 @@ impl Dialect for SnowflakeDialect {
             // Snowflake supports both `BEGIN TRANSACTION` and `BEGIN ... END` blocks.
             // If the next keyword indicates a transaction statement, let the
             // standard parse_begin() handle it.
-            if parser.peek_keyword(Keyword::TRANSACTION)
-                || parser.peek_keyword(Keyword::WORK)
-                || parser.peek_keyword(Keyword::NAME)
-                || parser.peek_token().token == Token::SemiColon
-                || parser.peek_token().token == Token::EOF
+            if parser
+                .peek_one_of_keywords(&[Keyword::TRANSACTION, Keyword::WORK, Keyword::NAME])
+                .is_some()
+                || matches!(parser.peek_token_ref().token, Token::SemiColon | Token::EOF)
             {
                 parser.prev_token();
                 return None;

tests/sqlparser_snowflake.rs

@@ -4626,6 +4626,9 @@ fn test_begin_transaction() {
         .parse_sql_statements("BEGIN; DROP TABLE IF EXISTS bla; COMMIT")
         .unwrap();
     assert_eq!(3, stmts.len());
+
+    // Bare BEGIN at EOF (no semicolon, no TRANSACTION keyword)
+    snowflake().verified_stmt("BEGIN");
 }
 
 #[test]