From 9975a0ffa89b4b3a61a3714905df326f11e9a28e Mon Sep 17 00:00:00 2001 From: Irit Katriel Date: Wed, 4 Jan 2023 12:14:36 +0000 Subject: [PATCH 1/2] gh-100747: some compiler macros use c instead of C to access the compiler --- Python/compile.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Python/compile.c b/Python/compile.c index cbbdfb9e946772..f2deada47a5cbd 100644 --- a/Python/compile.c +++ b/Python/compile.c @@ -1626,7 +1626,7 @@ cfg_builder_addop_j(cfg_builder *g, location loc, #define ADDOP_IN_SCOPE(C, LOC, OP) { \ if (cfg_builder_addop_noarg(CFG_BUILDER(C), (OP), (LOC)) < 0) { \ - compiler_exit_scope(c); \ + compiler_exit_scope(C); \ return -1; \ } \ } @@ -1692,7 +1692,7 @@ cfg_builder_addop_j(cfg_builder *g, location loc, #define VISIT_IN_SCOPE(C, TYPE, V) {\ if (compiler_visit_ ## TYPE((C), (V)) < 0) { \ - compiler_exit_scope(c); \ + compiler_exit_scope(C); \ return ERROR; \ } \ } @@ -1713,7 +1713,7 @@ cfg_builder_addop_j(cfg_builder *g, location loc, for (_i = 0; _i < asdl_seq_LEN(seq); _i++) { \ TYPE ## _ty elt = (TYPE ## _ty)asdl_seq_GET(seq, _i); \ if (compiler_visit_ ## TYPE((C), elt) < 0) { \ - compiler_exit_scope(c); \ + compiler_exit_scope(C); \ return ERROR; \ } \ } \ From 279af687deb87eac3ed66ea3de92703226864716 Mon Sep 17 00:00:00 2001 From: Irit Katriel Date: Wed, 4 Jan 2023 12:20:59 +0000 Subject: [PATCH 2/2] while we're here, fix a couple of macros to use the same style --- Python/compile.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Python/compile.c b/Python/compile.c index f2deada47a5cbd..e07de38ab5df0b 100644 --- a/Python/compile.c +++ b/Python/compile.c @@ -131,9 +131,9 @@ (opcode) == STORE_FAST__LOAD_FAST || \ (opcode) == STORE_FAST__STORE_FAST) -#define IS_TOP_LEVEL_AWAIT(c) ( \ - (c->c_flags.cf_flags & PyCF_ALLOW_TOP_LEVEL_AWAIT) \ - && (c->u->u_ste->ste_type == ModuleBlock)) +#define IS_TOP_LEVEL_AWAIT(C) ( \ + ((C)->c_flags.cf_flags & PyCF_ALLOW_TOP_LEVEL_AWAIT) \ + && ((C)->u->u_ste->ste_type == ModuleBlock)) typedef _PyCompilerSrcLocation location; @@ -479,7 +479,7 @@ struct compiler { PyArena *c_arena; /* pointer to memory allocation arena */ }; -#define CFG_BUILDER(c) (&((c)->u->u_cfg_builder)) +#define CFG_BUILDER(C) (&((C)->u->u_cfg_builder)) typedef struct {