[mw-devel] MARVIN r89 - branches/smonkey

pwb at sucs.org pwb at sucs.org
Sat Dec 16 23:00:11 GMT 2006


Author: pwb
Date: 2006-12-16 23:00:11 +0000 (Sat, 16 Dec 2006)
New Revision: 89

Modified:
   branches/smonkey/Makefile
Log:
* -Wextra is better than -W (it's the same, but is more descriptive)
* fixed a spelling error
* added explanation of a couple of automatic variables


Modified: branches/smonkey/Makefile
===================================================================
--- branches/smonkey/Makefile	2006-11-30 12:28:11 UTC (rev 88)
+++ branches/smonkey/Makefile	2006-12-16 23:00:11 UTC (rev 89)
@@ -9,17 +9,17 @@
 
 # CFLAGS is the standard variable containing the compiler's commandline options
 # The options we set are:
-#	-std=gnu99	- Tell the compiler that the code is GNU99 complient C
+#	-std=gnu99	- Tell the compiler that the code is GNU99 compliant C
 #			  (like ISO C99 but with some extra syntax)
 #	-g		- Add debugging symbols so we can debug the code in gdb
 #	-ggdb		- Add more debugging symbols for gdb
 #	-O2		- Enable code optimisations
 #	-Wall		- Enable compiler warnings
 #	-pedantic	- Enable some more compiler warnings
-#	-W		- Enable yet more compiler warnings :)
+#	-Wextra		- Enable yet more compiler warnings :)
 #	-fPIC		- Enable position independent code (for PowerPC?)
 # We also include the DEFS variable we defined above
-CFLAGS		= -std=gnu99 -g -O2 -Wall -pedantic -W -ggdb -fPIC $(DEFS)
+CFLAGS		= -std=gnu99 -g -O2 -Wall -pedantic -Wextra -ggdb -fPIC $(DEFS)
 LDFLAGS 	= -g
 
 # The names of all object files that make up the final binaries.  Just take the
@@ -57,7 +57,8 @@
 	@rm -vf $(ALLOBJS) $(BINARIES) *.d
 
 # A rule to tell it how to link all the object files together to make the
-# final "server" executable.
+# final "server" executable. $^ means all the dependencies listed, and $@ is
+# the target name.
 server:	$(SERVEROBJS)
 	$(CC) $(LDFLAGS) $(LIBS) $^ -o $@
 





More information about the mw-devel mailing list