diff options
author | Daniel Stenberg <daniel@haxx.se> | 2007-02-17 21:54:17 +0000 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2007-02-17 21:54:17 +0000 |
commit | 20a5926d2e0fe2cf7c5508f490ae04d61bfd07a4 (patch) | |
tree | 2dabff846860b975f6cb2d3e49567c72b0383be2 | |
parent | 5d45ec90e0614a96367e73d96fed8f47b112eced (diff) | |
download | rockbox-20a5926d2e0fe2cf7c5508f490ae04d61bfd07a4.tar.gz rockbox-20a5926d2e0fe2cf7c5508f490ae04d61bfd07a4.zip |
build with -Wundef to make sure we have spelled the defines right in #if lines
and have the proper includes included etc
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@12360 a1c6a512-1295-4272-9138-f99709370657
-rw-r--r-- | apps/codecs/libfaad/common.h | 20 | ||||
-rw-r--r-- | apps/debug_menu.c | 3 | ||||
-rw-r--r-- | apps/dsp.c | 3 | ||||
-rw-r--r-- | apps/gui/list.c | 2 | ||||
-rw-r--r-- | apps/screen_access.c | 2 | ||||
-rw-r--r-- | apps/screen_access.h | 2 | ||||
-rw-r--r-- | firmware/target/arm/usb-pp.c | 2 | ||||
-rwxr-xr-x | tools/configure | 2 |
8 files changed, 17 insertions, 19 deletions
diff --git a/apps/codecs/libfaad/common.h b/apps/codecs/libfaad/common.h index 65d27338ae..59d96252e5 100644 --- a/apps/codecs/libfaad/common.h +++ b/apps/codecs/libfaad/common.h | |||
@@ -181,33 +181,33 @@ typedef float float32_t; | |||
181 | #else | 181 | #else |
182 | 182 | ||
183 | #include <stdio.h> | 183 | #include <stdio.h> |
184 | #if HAVE_SYS_TYPES_H | 184 | #ifdef HAVE_SYS_TYPES_H |
185 | # include <sys/types.h> | 185 | # include <sys/types.h> |
186 | #endif | 186 | #endif |
187 | #if HAVE_SYS_STAT_H | 187 | #ifdef HAVE_SYS_STAT_H |
188 | # include <sys/stat.h> | 188 | # include <sys/stat.h> |
189 | #endif | 189 | #endif |
190 | #if STDC_HEADERS | 190 | #ifdef STDC_HEADERS |
191 | # include <stdlib.h> | 191 | # include <stdlib.h> |
192 | # include <stddef.h> | 192 | # include <stddef.h> |
193 | #else | 193 | #else |
194 | # if HAVE_STDLIB_H | 194 | # ifdef HAVE_STDLIB_H |
195 | # include <stdlib.h> | 195 | # include <stdlib.h> |
196 | # endif | 196 | # endif |
197 | #endif | 197 | #endif |
198 | #if HAVE_STRING_H | 198 | #ifdef HAVE_STRING_H |
199 | # if !STDC_HEADERS && HAVE_MEMORY_H | 199 | # if !defined(STDC_HEADERS) && defined(HAVE_MEMORY_H) |
200 | # include <memory.h> | 200 | # include <memory.h> |
201 | # endif | 201 | # endif |
202 | //# include <string.h> | 202 | //# include <string.h> |
203 | #endif | 203 | #endif |
204 | #if HAVE_STRINGS_H | 204 | #ifdef HAVE_STRINGS_H |
205 | //# include <strings.h> | 205 | //# include <strings.h> |
206 | #endif | 206 | #endif |
207 | #if HAVE_INTTYPES_H | 207 | #ifdef HAVE_INTTYPES_H |
208 | # include <inttypes.h> | 208 | # include <inttypes.h> |
209 | #else | 209 | #else |
210 | # if HAVE_STDINT_H | 210 | # ifdef HAVE_STDINT_H |
211 | # include <stdint.h> | 211 | # include <stdint.h> |
212 | # else | 212 | # else |
213 | /* we need these... */ | 213 | /* we need these... */ |
@@ -221,7 +221,7 @@ typedef short int16_t; | |||
221 | typedef char int8_t; | 221 | typedef char int8_t; |
222 | # endif | 222 | # endif |
223 | #endif | 223 | #endif |
224 | #if HAVE_UNISTD_H | 224 | #ifdef HAVE_UNISTD_H |
225 | # include <unistd.h> | 225 | # include <unistd.h> |
226 | #endif | 226 | #endif |
227 | 227 | ||
diff --git a/apps/debug_menu.c b/apps/debug_menu.c index 98cbf13860..95b9aa8f01 100644 --- a/apps/debug_menu.c +++ b/apps/debug_menu.c | |||
@@ -944,9 +944,6 @@ static bool dbg_spdif(void) | |||
944 | #elif CONFIG_KEYPAD == ONDIO_PAD | 944 | #elif CONFIG_KEYPAD == ONDIO_PAD |
945 | # define DEBUG_CANCEL BUTTON_MENU | 945 | # define DEBUG_CANCEL BUTTON_MENU |
946 | 946 | ||
947 | #elif CONFIG_KEYPAD == GMINI100_PAD | ||
948 | # define DEBUG_CANCEL BUTTON_OFF | ||
949 | |||
950 | #elif (CONFIG_KEYPAD == IPOD_3G_PAD) || \ | 947 | #elif (CONFIG_KEYPAD == IPOD_3G_PAD) || \ |
951 | (CONFIG_KEYPAD == IPOD_4G_PAD) | 948 | (CONFIG_KEYPAD == IPOD_4G_PAD) |
952 | # define DEBUG_CANCEL BUTTON_MENU | 949 | # define DEBUG_CANCEL BUTTON_MENU |
diff --git a/apps/dsp.c b/apps/dsp.c index 02e231d800..7ecb791f50 100644 --- a/apps/dsp.c +++ b/apps/dsp.c | |||
@@ -16,10 +16,11 @@ | |||
16 | * KIND, either express or implied. | 16 | * KIND, either express or implied. |
17 | * | 17 | * |
18 | ****************************************************************************/ | 18 | ****************************************************************************/ |
19 | #include "config.h" | ||
20 | |||
19 | #include <inttypes.h> | 21 | #include <inttypes.h> |
20 | #include <string.h> | 22 | #include <string.h> |
21 | #include <sound.h> | 23 | #include <sound.h> |
22 | #include "config.h" | ||
23 | #include "dsp.h" | 24 | #include "dsp.h" |
24 | #include "eq.h" | 25 | #include "eq.h" |
25 | #include "kernel.h" | 26 | #include "kernel.h" |
diff --git a/apps/gui/list.c b/apps/gui/list.c index 6d966c23ae..67b34e5d95 100644 --- a/apps/gui/list.c +++ b/apps/gui/list.c | |||
@@ -812,7 +812,7 @@ unsigned gui_synclist_do_button(struct gui_synclist * lists, | |||
812 | gui_synclist_draw(lists); | 812 | gui_synclist_draw(lists); |
813 | yield(); | 813 | yield(); |
814 | return ACTION_STD_PREV; | 814 | return ACTION_STD_PREV; |
815 | #if (REMOTE_BUTTON != 0 ) | 815 | #ifdef REMOTE_BUTTON |
816 | case ACTION_LISTTREE_RC_PGUP: | 816 | case ACTION_LISTTREE_RC_PGUP: |
817 | gui_synclist_select_previous_page(lists, SCREEN_REMOTE); | 817 | gui_synclist_select_previous_page(lists, SCREEN_REMOTE); |
818 | gui_synclist_draw(lists); | 818 | gui_synclist_draw(lists); |
diff --git a/apps/screen_access.c b/apps/screen_access.c index b784637308..e2714b2324 100644 --- a/apps/screen_access.c +++ b/apps/screen_access.c | |||
@@ -140,7 +140,7 @@ void screen_init(struct screen * screen, enum screen_type screen_type) | |||
140 | screen->transparent_bitmap=&lcd_bitmap_transparent; | 140 | screen->transparent_bitmap=&lcd_bitmap_transparent; |
141 | screen->transparent_bitmap_part=&lcd_bitmap_transparent_part; | 141 | screen->transparent_bitmap_part=&lcd_bitmap_transparent_part; |
142 | #endif | 142 | #endif |
143 | #if defined(HAVE_LCD_COLOR) && LCD_REMOTE_DEPTH > 1 | 143 | #if defined(HAVE_LCD_COLOR) && defined(LCD_REMOTE_DEPTH) && LCD_REMOTE_DEPTH > 1 |
144 | screen->color_to_native=&lcd_color_to_native; | 144 | screen->color_to_native=&lcd_color_to_native; |
145 | #endif | 145 | #endif |
146 | screen->get_background=&lcd_get_background; | 146 | screen->get_background=&lcd_get_background; |
diff --git a/apps/screen_access.h b/apps/screen_access.h index cf74a97ba1..eea15e86a5 100644 --- a/apps/screen_access.h +++ b/apps/screen_access.h | |||
@@ -93,7 +93,7 @@ struct screen | |||
93 | void (*transparent_bitmap_part)(const fb_data *src, int src_x, int src_y, | 93 | void (*transparent_bitmap_part)(const fb_data *src, int src_x, int src_y, |
94 | int stride, int x, int y, int width, int height); | 94 | int stride, int x, int y, int width, int height); |
95 | void (*set_drawmode)(int mode); | 95 | void (*set_drawmode)(int mode); |
96 | #if defined(HAVE_LCD_COLOR) && LCD_REMOTE_DEPTH > 1 | 96 | #if defined(HAVE_LCD_COLOR) && defined(LCD_REMOTE_DEPTH) && LCD_REMOTE_DEPTH > 1 |
97 | unsigned (*color_to_native)(unsigned color); | 97 | unsigned (*color_to_native)(unsigned color); |
98 | #endif | 98 | #endif |
99 | #if (LCD_DEPTH > 1) || (LCD_REMOTE_DEPTH > 1) | 99 | #if (LCD_DEPTH > 1) || (LCD_REMOTE_DEPTH > 1) |
diff --git a/firmware/target/arm/usb-pp.c b/firmware/target/arm/usb-pp.c index d69c6e6c9c..b497e86804 100644 --- a/firmware/target/arm/usb-pp.c +++ b/firmware/target/arm/usb-pp.c | |||
@@ -96,7 +96,7 @@ void usb_enable(bool on) | |||
96 | handling, which should be the same for all PortalPlayer targets. */ | 96 | handling, which should be the same for all PortalPlayer targets. */ |
97 | if (on) | 97 | if (on) |
98 | { | 98 | { |
99 | #if IPOD_ARCH || defined(IRIVER_H10) || defined (IRIVER_H10_5GB) | 99 | #if defined(IPOD_ARCH) || defined(IRIVER_H10) || defined (IRIVER_H10_5GB) |
100 | /* For the H10 and iPod, we can only do one thing with USB mode - reboot | 100 | /* For the H10 and iPod, we can only do one thing with USB mode - reboot |
101 | into the flash-based disk-mode. This does not return. */ | 101 | into the flash-based disk-mode. This does not return. */ |
102 | 102 | ||
diff --git a/tools/configure b/tools/configure index 77884ef02c..f2626492fb 100755 --- a/tools/configure +++ b/tools/configure | |||
@@ -9,7 +9,7 @@ | |||
9 | # | 9 | # |
10 | 10 | ||
11 | # global CC options for all platforms | 11 | # global CC options for all platforms |
12 | CCOPTS="-W -Wall -O -nostdlib -ffreestanding -Wstrict-prototypes" | 12 | CCOPTS="-W -Wall -Wundef -O -nostdlib -ffreestanding -Wstrict-prototypes" |
13 | 13 | ||
14 | use_logf="#undef ROCKBOX_HAS_LOGF" | 14 | use_logf="#undef ROCKBOX_HAS_LOGF" |
15 | use_simsound="#undef ROCKBOX_HAS_SIMSOUND" | 15 | use_simsound="#undef ROCKBOX_HAS_SIMSOUND" |