Added -Wextra to compiler flags, found out about redundant test.

diff --git a/Makefile b/Makefile
index c7831bd86e01a79c5abc0a449d2efba192eceff0..3280dd35c31aec8b2791fd23586e0bd9ed76cf0c 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,5 +1,5 @@
 CC=gcc
-CFLAGS=-Wall -g
+CFLAGS=-Wall -Wextra -g
 TARGET_SERVER=roguelike-server
 TARGET_CLIENT=roguelike-client
 SRCDIR=src

diff --git a/src/server/map.c b/src/server/map.c
index 8bae81a4a47b159233fbeb8d6a73fe6d296b06ab..0edd955c4265419f7f09d19d14e10c1066384a62 100644 (file)
--- a/src/server/map.c
+++ b/src/server/map.c
@@ -55,8 +55,7 @@ extern void init_map()
 extern uint8_t is_passable(struct yx_uint8 pos)
 {
     uint8_t passable = 0;
-    if (   0 <= pos.x && pos.x < world.map.size.x
-        && 0 <= pos.y && pos.y < world.map.size.y)
+    if (pos.x < world.map.size.x && pos.y < world.map.size.y)
     {
         passable = ('.' == world.map.cells[(pos.y * world.map.size.x) + pos.x]);
     }