Added a comment explaining the extern ugliness.

diff --git a/Modules/cjkcodecs/cjkcodecs.h b/Modules/cjkcodecs/cjkcodecs.h
index 6fa8a7b1b77f0c4d022196c7e0967489cadfb03b..bf08e588fd62db813f2f5f9243b0acc883494171 100644 (file)
--- a/Modules/cjkcodecs/cjkcodecs.h
+++ b/Modules/cjkcodecs/cjkcodecs.h
@@ -57,6 +57,12 @@ struct pair_encodemap {
        DBCHAR code;
 };
 
+/* There are really static, and (re)declared so later by the expansions
+ * of the BEGIN_MAPPINGS_LIST and BEGIN_CODECS_LIST macros, but it's
+ * not legal C to declare a static array of unknown size.  It would be
+ * better if the code were rearranged so as to not require declaration
+ * of these names before the macros define them.
+ */
 extern const MultibyteCodec codec_list[];
 extern const struct dbcs_map mapping_list[];