This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author skip.montanaro
Recipients pablogsal, skip.montanaro
Date 2020-09-23.13:09:08
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CANc-5UzGFL340qFXmwScNyZBRjzUKBdgZqnQQLi8TyWs=KqRnw@mail.gmail.com>
In-reply-to <1600715216.59.0.0182711956873.issue41826@roundup.psfhosted.org>
Content
> This is because the types of the grammars in
Lib/test/test_peg_generator/test_c_parser.py needs to be updated to use the
new sequences

I'm sorry, that doesn't mean much to me. Is it simply a matter of applying
suitable casts to the various parameters in the C code fragments in those?
For example:

*** /tmp/ediffGoXpTW 2020-09-23 08:06:38.671061988 -0500
---
/home/skip/src/python/cpython/Lib/test/test_peg_generator/test_c_parser.py
2020-09-21 15:54:43.106258797 -0500
***************
*** 325,331 ****
          compound_stmt[stmt_ty]: with_stmt
          with_stmt[stmt_ty]: (
              a='with' '(' b=','.with_item+ ')' ':' c=block {
!                 _Py_With(b, _PyPegen_singleton_seq(p, c), NULL, EXTRA) }
          )
          with_item[withitem_ty]: (
              e=NAME o=['as' t=NAME { t }] { _Py_withitem(e,
_PyPegen_set_expr_context(p, o, Store), p->arena) }
--- 325,331 ----
          compound_stmt[stmt_ty]: with_stmt
          with_stmt[stmt_ty]: (
              a='with' '(' b=','.with_item+ ')' ':' c=block {
!                 _Py_With((asdl_withitem_seq *)b, *(asdl_stmt_seq
*)*_PyPegen_singleton_seq(p,
c), NULL, EXTRA) }
          )
          with_item[withitem_ty]: (
              e=NAME o=['as' t=NAME { t }] { _Py_withitem(e,
_PyPegen_set_expr_context(p, o, Store), p->arena) }

If so, that's pretty mechanical. Even a dummy like me could produce a patch.
History
Date User Action Args
2020-09-23 13:09:09skip.montanarosetrecipients: + skip.montanaro, pablogsal
2020-09-23 13:09:09skip.montanarolinkissue41826 messages
2020-09-23 13:09:08skip.montanarocreate