Mercurial > sdl-ios-xcode
view Makefile.minimal @ 1626:a80e1e0880b8
Fixed bug #176
[I'm fixing this for the public headers, but I'm not going to bother for the SDL library code (yet)]
To clarify: Normaly, GCC (or, to be precise, the preprocessor) will ignore
this, and compile the code happily. However, one can specify -Wundef to get a
warning about this.
One can probably argue whether to consider this a bug or not; but I think that
(a) from a semantic point of view, using "#if FOO" when FOO is not defined is
strange, and (b) since it is possible to trigger a warning about this, and a
trivial fix exists, it should be corrected.
I can think of two alternative patches, BTW:
1) Simply use #define HAVE_FOO 0, instead of not defining HAVE_FOO at all
2) Change
#if HAVE_FOO
to
#if HAVE_FOO+0
which always does the right thing.
But I think I still prefer the attached patch :-).
author | Sam Lantinga <slouken@libsdl.org> |
---|---|
date | Thu, 13 Apr 2006 13:38:40 +0000 |
parents | 64710b9f3eeb |
children | a72a5f62d6b7 |
line wrap: on
line source
# Makefile to build the SDL library INCLUDE = -I./include CFLAGS = -g -O2 $(INCLUDE) AR = ar RANLIB = ranlib CONFIG_H = include/SDL_config.h TARGET = libSDL.a SOURCES = \ src/*.c \ src/audio/*.c \ src/cdrom/*.c \ src/cpuinfo/*.c \ src/events/*.c \ src/file/*.c \ src/joystick/*.c \ src/stdlib/*.c \ src/thread/*.c \ src/timer/*.c \ src/video/*.c \ src/audio/dummy/*.c \ src/video/dummy/*.c \ src/joystick/dummy/*.c \ src/cdrom/dummy/*.c \ src/thread/generic/*.c \ src/timer/dummy/*.c \ src/loadso/dummy/*.c \ OBJECTS = $(shell echo $(SOURCES) | sed -e 's,\.c,\.o,g') all: $(TARGET) $(TARGET): $(CONFIG_H) $(OBJECTS) $(AR) crv $@ $^ $(RANLIB) $@ $(CONFIG_H): cp $(CONFIG_H).default $(CONFIG_H) clean: rm -f $(TARGET) $(OBJECTS)