:: commit d9b062917a8b44eb4508828f8a15e144c05e6a93

mintsuki <mintsuki@protonmail.com> — 2020-10-01 00:16

parents: d9e40a7dff

Fix all warnings and add -Werror to CFLAGS so people stop committing broken code

diff --git a/decompressor/Makefile b/decompressor/Makefile
index 642bca50..108b5b20 100644
--- a/decompressor/Makefile
+++ b/decompressor/Makefile
@@ -2,7 +2,7 @@ CC = i386-elf-gcc
 LD = i386-elf-gcc
 OBJCOPY = i386-elf-objcopy
 
-CFLAGS = -flto -Os -pipe -Wall -Wextra
+CFLAGS = -flto -Os -pipe -Wall -Wextra -Werror
 
 INTERNAL_CFLAGS = \
 	-std=gnu11 \
diff --git a/stage2/Makefile b/stage2/Makefile
index f2051cfe..90f4cff9 100644
--- a/stage2/Makefile
+++ b/stage2/Makefile
@@ -2,7 +2,7 @@ CC = i386-elf-gcc
 LD = i386-elf-gcc
 OBJCOPY = i386-elf-objcopy
 
-CFLAGS = -flto -Os -pipe -Wall -Wextra
+CFLAGS = -flto -Os -pipe -Wall -Wextra -Werror
 
 INTERNAL_CFLAGS = \
 	-std=gnu11 \
diff --git a/stage2/drivers/vga_textmode.c b/stage2/drivers/vga_textmode.c
index b7c7a2eb..6165cc84 100644
--- a/stage2/drivers/vga_textmode.c
+++ b/stage2/drivers/vga_textmode.c
@@ -74,29 +74,6 @@ void init_vga_textmode(int *_rows, int *_cols) {
     *_cols = VD_COLS / 2;
 }
 
-static void text_set_cursor_palette(uint8_t c) {
-    cursor_palette = c;
-    draw_cursor();
-    return;
-}
-
-static uint8_t text_get_cursor_palette(void) {
-    return cursor_palette;
-}
-
-static void text_set_text_palette(uint8_t c) {
-    text_palette = c;
-    return;
-}
-
-static uint8_t text_get_text_palette(void) {
-    return text_palette;
-}
-
-static int text_get_cursor_pos_x(void) {
-    return (cursor_offset % VD_COLS) / 2;
-}
-
 static int text_get_cursor_pos_y(void) {
     return cursor_offset / VD_COLS;
 }
tab: 248 wrap: offon