From: Pablo Galindo Salgado <Pablogsal@gmail.com>
Date: Tue, 18 Jan 2022 11:13:00 +0000 (+0000)
Subject: bpo-46339: Include clarification on assert in 'get_error_line_from_tokenizer_buffers... 
X-Git-Tag: v3.11.0a5~223
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=8c2fd09f365e082cfceb29afdf38953cdd670946;p=thirdparty%2FPython%2Fcpython.git

bpo-46339: Include clarification on assert in 'get_error_line_from_tokenizer_buffers' (#30545)
---

diff --git a/Parser/pegen_errors.c b/Parser/pegen_errors.c
index bffae8532ca2..f348ac3000dd 100644
--- a/Parser/pegen_errors.c
+++ b/Parser/pegen_errors.c
@@ -254,6 +254,9 @@ get_error_line_from_tokenizer_buffers(Parser *p, Py_ssize_t lineno)
 
     for (int i = 0; i < relative_lineno - 1; i++) {
         char *new_line = strchr(cur_line, '\n') + 1;
+        // The assert is here for debug builds but the conditional that
+        // follows is there so in release builds we do not crash at the cost
+        // to report a potentially wrong line.
         assert(new_line != NULL && new_line < p->tok->inp);
         if (new_line == NULL || new_line >= p->tok->inp) {
             break;