osc-osd: Make all warnings Errors and have a clean build

* Add the -Werror to the flags. We need a clean compilation * Fix a warning in get_mhz() which was actually an error. good catch gcc, thanks Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
SNSL · Nov 3, 2011 · f8d1b45 · f8d1b45
1 parent f2378b4
commit f8d1b45
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/osd-target/Makefile b/osd-target/Makefile
@@ -45,7 +45,7 @@ COPTS := $(OPT)
 COPTS += -fPIC -foptimize-sibling-calls -D_GNU_SOURCE
 CWARN := -Wall -W -Wpointer-arith -Wwrite-strings -Wcast-align -Wcast-qual \
 	-Wbad-function-cast -Wundef -Wmissing-prototypes \
-	-Wmissing-declarations -Wnested-externs
+	-Wmissing-declarations -Wnested-externs -Werror
 CFLAGS += $(COPTS) $(CWARN) -I.. -Wno-unused -I../osd-util
 
 ifeq ($(PVFS_OSD_INTEGRATED),1)

diff --git a/osd-util/osd-util.c b/osd-util/osd-util.c
@@ -693,7 +693,7 @@ double get_mhz(void)
 	fclose(fp); 
 #endif /*  else defined(__FreeBSD__) || defined(__APPLE__) */
 
-	return mhz != 0 ?: cpufrequency_not_found ;
+	return mhz != 0 ? mhz : cpufrequency_not_found ;
 }
 
 /*