Move LONG_BIT from intobject.c to pyport.h. #error if it's already been

tim-one · tim-one · commit d57731f74bda · 2000-10-05T01:42:25.000Z
#define'd to an unreasonable value (several recent gcc systems have
misdefined it, causing bogus overflows in integer multiplication).  Nuke
CHAR_BIT entirely.
diff --git a/Include/pyport.h b/Include/pyport.h
@@ -374,6 +374,19 @@ typedef	struct fd_set {
 #define LONG_MIN (-LONG_MAX-1)
 #endif
 
+#ifndef LONG_BIT
+#define LONG_BIT (8 * SIZEOF_LONG)
+#endif
+
+#if LONG_BIT != 8 * SIZEOF_LONG
+/* 04-Oct-2000 LONG_BIT is apparently (mis)defined as 64 on some recent
+ * 32-bit platforms using gcc.  We try to catch that here at compile-time
+ * rather than waiting for integer multiplication to trigger bogus
+ * overflows.
+ */
+#error "LONG_BIT definition appears wrong for platform (bad gcc config?)."
+#endif
+
 #ifdef __NeXT__
 #ifdef __sparc__
 /*
diff --git a/Objects/intobject.c b/Objects/intobject.c
@@ -4,14 +4,6 @@
 #include "Python.h"
 #include <ctype.h>
 
-#ifndef CHAR_BIT
-#define CHAR_BIT 8
-#endif
-
-#ifndef LONG_BIT
-#define LONG_BIT (CHAR_BIT * sizeof(long))
-#endif
-
 long
 PyInt_GetMax(void)
 {
