René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Check tokens[0] after allocating memory","articleBody":"\u003c!--\r\n  If you're new to Python and you're not sure whether what you're experiencing is a bug, the CPython issue tracker is not\r\n  the right place to seek help. Consider the following options instead:\r\n\r\n  - reading the Python tutorial: https://docs.python.org/3/tutorial/\r\n  - posting in the \"Users\" category on discuss.python.org: https://discuss.python.org/c/users/7\r\n  - emailing the Python-list mailing list: https://mail.python.org/mailman/listinfo/python-list\r\n  - searching our issue tracker (https://github.com/python/cpython/issues) to see if\r\n    your problem has already been reported\r\n--\u003e\r\n\r\n# Bug report\r\n\r\nI have questions when reading the source code of file `pegen.c`.\r\nThere is a code snippet in function `_PyPegen_Parser_New`:\r\n```c\r\np-\u003etokens[0] = PyMem_Calloc(1, sizeof(Token));\r\nif (!p-\u003etokens) {\r\n    PyMem_Free(p-\u003etokens);\r\n    PyMem_Free(p);\r\n    return (Parser *) PyErr_NoMemory();\r\n}\r\n```\r\nI think it makes more sense to check `p-\u003etokens[0]` but not `p-\u003etokens` in if condition.\r\n\r\nI look at the PR #19669 where the code was introduced and no one discussed this. Is this an oversight, or am I wrong?\r\n\r\n# Your environment\r\n\r\n\u003c!-- Include as many relevant details as possible about the environment you experienced the bug in --\u003e\r\n\r\n- CPython versions tested on: main branch\r\n\r\n\u003c!--\r\nYou can freely edit this text. Remove any lines you believe are unnecessary.\r\n--\u003e\r\n","author":{"url":"https://github.com/imzhuhl","@type":"Person","name":"imzhuhl"},"datePublished":"2022-07-28T05:08:16.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/95355/cpython/issues/95355"}