Add more strict checking to C library compile

ghaerr · Nov 20, 2024 · 6744070 · 6744070
1 parent 3681e32
commit 6744070
Show file tree

Hide file tree

Showing 9 changed files with 26 additions and 9 deletions.
diff --git a/libc/ia16.inc b/libc/ia16.inc
@@ -19,6 +19,8 @@ AR=ia16-elf-ar
 LD=ia16-elf-ld
 
 CFLAGS=$(ARCH) $(INCLUDES) $(CDEFS) -Wall -Os $(MULTILIB)
+CFLAGS+=-Wextra -Wtype-limits -Wno-unused-parameter -Wno-sign-compare
+CFLAGS+=$(MULTILIB)
 ASFLAGS=--32-segelf -mtune=i8086
 LDFLAGS=-mtune=i8086
 # Used in subdirs to quickly create a library archive without a symbol index

diff --git a/libc/math/Makefile b/libc/math/Makefile
@@ -5,7 +5,7 @@ LIB ?= out.a
 
 include $(TOPDIR)/libc/$(COMPILER).inc
 
-CFLAGS += -I$(TOPDIR)/libc/math -D__BSD_VISIBLE
+CFLAGS += -I$(TOPDIR)/libc/math -D__BSD_VISIBLE -Wno-type-limits
 
 DOUBLE = \
 	s_floor.o \

diff --git a/libc/stdio/sprintf.c b/libc/stdio/sprintf.c
@@ -12,7 +12,10 @@ sprintf(char *sp, const char *fmt, ...)
      0,
      (unsigned char *)(unsigned)-1,
      -1,
-     _IOFBF | __MODE_WRITE}
+     _IOFBF | __MODE_WRITE,
+     { 0,0,0,0,0,0,0,0 },
+     0
+    }
   };
 
   va_list ptr;

diff --git a/libc/stdio/sscanf.c b/libc/stdio/sscanf.c
@@ -12,7 +12,9 @@ sscanf(const char *str, const char *format, ...)
 			0,
 			(unsigned char *)(unsigned)-1,
 			-1,
-			_IOFBF | __MODE_READ
+			_IOFBF | __MODE_READ,
+			{ 0,0,0,0,0,0,0,0 },
+			0
 		}
 	};
 

diff --git a/libc/stdio/stderr.c b/libc/stdio/stderr.c
@@ -13,6 +13,8 @@ FILE  stderr[1] =
     buferr,
     buferr + sizeof(buferr),
     2,
-    _IONBF | __MODE_WRITE | __MODE_IOTRAN
+    _IONBF | __MODE_WRITE | __MODE_IOTRAN,
+    { 0,0,0,0,0,0,0,0 },
+    0
    }
 };
diff --git a/libc/stdio/stdin.c b/libc/stdio/stdin.c
@@ -11,6 +11,8 @@ FILE  stdin[1] =
     bufin,
     bufin + sizeof(bufin),
     0,
-    _IOFBF | __MODE_READ | __MODE_IOTRAN
+    _IOFBF | __MODE_READ | __MODE_IOTRAN,
+    { 0,0,0,0,0,0,0,0 },
+    0
    }
 };
diff --git a/libc/stdio/stdout.c b/libc/stdio/stdout.c
@@ -12,9 +12,11 @@ FILE  stdout[1] =
     bufout + sizeof(bufout),
     1,
 #ifdef __WATCOMC__
-    _IOLBF | __MODE_WRITE | __MODE_IOTRAN   /* FIXME flush on exit to fix */
+    _IOLBF | __MODE_WRITE | __MODE_IOTRAN,  /* FIXME flush on exit to fix */
 #else
-    _IOFBF | __MODE_WRITE | __MODE_IOTRAN
+    _IOFBF | __MODE_WRITE | __MODE_IOTRAN,
 #endif
+    { 0,0,0,0,0,0,0,0 },
+    0
    }
 };
diff --git a/libc/stdio/vsnprintf.c b/libc/stdio/vsnprintf.c
@@ -12,7 +12,9 @@ vsnprintf(char *sp, size_t size, const char *fmt, va_list ap)
 			0,
 			(unsigned char *)(unsigned)-1,
 			-1,
-		 _IOFBF | __MODE_WRITE
+			_IOFBF | __MODE_WRITE,
+			{ 0,0,0,0,0,0,0,0 },
+			0
 		 }
 	};
 

diff --git a/libc/stdio/vsprintf.c b/libc/stdio/vsprintf.c
@@ -12,7 +12,9 @@ vsprintf(char *sp, const char *fmt, va_list ap)
 			0,
 			(unsigned char *)(unsigned)-1,
 			-1,
-		 _IOFBF | __MODE_WRITE
+			_IOFBF | __MODE_WRITE,
+			{ 0,0,0,0,0,0,0,0 },
+			0
 		 }
 	};