]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.13] gh-130077: Properly match full soft keywords in the parser (GH-135317) (#135399)
authorPablo Galindo Salgado <Pablogsal@gmail.com>
Mon, 4 Aug 2025 14:18:05 +0000 (15:18 +0100)
committerGitHub <noreply@github.com>
Mon, 4 Aug 2025 14:18:05 +0000 (15:18 +0100)
* [3.13] gh-130077: Properly match full soft keywords in the parser (GH-135317)
(cherry picked from commit ff2b5f40c2bf5c71255caac8a743c09ba0758c02)

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
* Remove line the main-branch commit removed

---------

Co-authored-by: Petr Viktorin <encukou@gmail.com>
Lib/test/test_syntax.py
Misc/NEWS.d/next/Core_and_Builtins/2025-06-09-23-57-37.gh-issue-130077.MHknDB.rst [new file with mode: 0644]
Parser/pegen.c

index 68dceac97b53dc79bd848d89682ef8b7e4b5f46d..b035883cf883b939933f272f6da94085a3c603a5 100644 (file)
@@ -352,6 +352,13 @@ SyntaxError: invalid syntax
 Traceback (most recent call last):
 SyntaxError: invalid syntax
 
+# But prefixes of soft keywords should
+# still raise specialized errors
+
+>>> (mat x)
+Traceback (most recent call last):
+SyntaxError: invalid syntax. Perhaps you forgot a comma?
+
 From compiler_complex_args():
 
 >>> def f(None=1):
diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2025-06-09-23-57-37.gh-issue-130077.MHknDB.rst b/Misc/NEWS.d/next/Core_and_Builtins/2025-06-09-23-57-37.gh-issue-130077.MHknDB.rst
new file mode 100644 (file)
index 0000000..a7d0242
--- /dev/null
@@ -0,0 +1,2 @@
+Properly raise custom syntax errors when incorrect syntax containing names
+that are prefixes of soft keywords is encountered.  Patch by Pablo Galindo.
index 656c42b348a96bde06ddbb23cde22cc3c79818c2..f01cc1ed03f287e23d876380a2fde11342c16b78 100644 (file)
@@ -609,7 +609,8 @@ expr_ty _PyPegen_soft_keyword_token(Parser *p) {
     Py_ssize_t size;
     PyBytes_AsStringAndSize(t->bytes, &the_token, &size);
     for (char **keyword = p->soft_keywords; *keyword != NULL; keyword++) {
-        if (strncmp(*keyword, the_token, size) == 0) {
+        if (strlen(*keyword) == (size_t)size &&
+            strncmp(*keyword, the_token, (size_t)size) == 0) {
             return _PyPegen_name_from_token(p, t);
         }
     }