Skip to content

Protect multi-line macros expansions in _test*.c files #120674

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
picnixz opened this issue Jun 18, 2024 · 1 comment
Closed

Protect multi-line macros expansions in _test*.c files #120674

picnixz opened this issue Jun 18, 2024 · 1 comment
Labels
type-feature A feature request or enhancement

Comments

@picnixz
Copy link
Member

picnixz commented Jun 18, 2024

Feature or enhancement

Proposal:

In Modules/_testbuffer.c and Modules/_testcapimodule.c, there are some expansions that are not protected, although their usage is correct.

This issue is in continuation with #119981 (comment) and #120017. There are other places where macro expansions are not protected, so I'll work on that when I've some free time (those PRs are quite easy to review and to implement IMO).

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Linked PRs

@vstinner
Copy link
Member

Fixed by 7c5da94.

@AlexWaygood AlexWaygood changed the title Protect multi-line macros exapsnions in _test*.c files Protect multi-line macros expansions in _test*.c files Jun 18, 2024
picnixz added a commit to picnixz/cpython that referenced this issue Jun 19, 2024
mrahtz pushed a commit to mrahtz/cpython that referenced this issue Jun 30, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this issue Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this issue Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

2 participants