Hide ugly code in macro

erlend-aasland · Apr 16, 2023 · 65af110 · 65af110
1 parent 605c4fb
commit 65af110
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/Modules/cjkcodecs/_codecs_iso2022.c b/Modules/cjkcodecs/_codecs_iso2022.c
@@ -1124,7 +1124,7 @@ BEGIN_MAPPINGS_LIST(0)
 END_MAPPINGS_LIST
 
 #define ISO2022_CODEC(variation)                \
-st->codec_list[idx++] = (MultibyteCodec){       \
+NEXT_CODEC = (MultibyteCodec){                  \
     "iso2022_" #variation,                      \
     &iso2022_##variation##_config,              \
     iso2022_codec_init,                         \

diff --git a/Modules/cjkcodecs/_codecs_jp.c b/Modules/cjkcodecs/_codecs_jp.c
@@ -748,7 +748,7 @@ BEGIN_MAPPINGS_LIST(11)
 END_MAPPINGS_LIST
 
 #define CODEC_CUSTOM(NAME, N, METH) \
-  st->codec_list[idx++] = (MultibyteCodec){NAME, (void *)N, NULL, _STATELESS_METHODS(METH)};
+    NEXT_CODEC = (MultibyteCodec){NAME, (void *)N, NULL, _STATELESS_METHODS(METH)};
 
 BEGIN_CODECS_LIST(7)
   CODEC_STATELESS(shift_jis)

diff --git a/Modules/cjkcodecs/cjkcodecs.h b/Modules/cjkcodecs/cjkcodecs.h
@@ -261,12 +261,15 @@ add_codecs(cjkcodecs_module_state *st)                          \
     enc##_encode, NULL, NULL,           \
     enc##_decode, NULL, NULL,
 
+#define NEXT_CODEC \
+    st->codec_list[idx++]
+
 #define CODEC_STATEFUL(enc) \
-    st->codec_list[idx++] = (MultibyteCodec){#enc, NULL, NULL, _STATEFUL_METHODS(enc)};
+    NEXT_CODEC = (MultibyteCodec){#enc, NULL, NULL, _STATEFUL_METHODS(enc)};
 #define CODEC_STATELESS(enc) \
-    st->codec_list[idx++] = (MultibyteCodec){#enc, NULL, NULL, _STATELESS_METHODS(enc)};
+    NEXT_CODEC = (MultibyteCodec){#enc, NULL, NULL, _STATELESS_METHODS(enc)};
 #define CODEC_STATELESS_WINIT(enc) \
-    st->codec_list[idx++] = (MultibyteCodec){#enc, NULL, enc##_codec_init, _STATELESS_METHODS(enc)};
+    NEXT_CODEC = (MultibyteCodec){#enc, NULL, enc##_codec_init, _STATELESS_METHODS(enc)};
 
 #define END_CODECS_LIST             \
     assert(st->num_codecs == idx);  \