Treat warnings as errors

TheLevelUp · Oct 24, 2014 · 50f71ca · 50f71ca
1 parent befbef7
commit 50f71ca
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/ZXingObjC.xcodeproj/project.pbxproj b/ZXingObjC.xcodeproj/project.pbxproj
@@ -5515,6 +5515,7 @@
 				COPY_PHASE_STRIP = YES;
 				FRAMEWORK_VERSION = 3.0.3;
 				GCC_C_LANGUAGE_STANDARD = gnu99;
+				GCC_TREAT_WARNINGS_AS_ERRORS = YES;
 				GCC_WARN_ABOUT_RETURN_TYPE = YES;
 				GCC_WARN_UNINITIALIZED_AUTOS = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;
@@ -5647,6 +5648,7 @@
 					"$(inherited)",
 				);
 				GCC_SYMBOLS_PRIVATE_EXTERN = NO;
+				GCC_TREAT_WARNINGS_AS_ERRORS = YES;
 				GCC_WARN_ABOUT_RETURN_TYPE = YES;
 				GCC_WARN_UNINITIALIZED_AUTOS = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;
@@ -5671,6 +5673,7 @@
 				COPY_PHASE_STRIP = YES;
 				FRAMEWORK_VERSION = 3.0.3;
 				GCC_C_LANGUAGE_STANDARD = gnu99;
+				GCC_TREAT_WARNINGS_AS_ERRORS = YES;
 				GCC_WARN_ABOUT_RETURN_TYPE = YES;
 				GCC_WARN_UNINITIALIZED_AUTOS = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;

diff --git a/ZXingObjC/client/ZXCapture.m b/ZXingObjC/client/ZXCapture.m
@@ -252,7 +252,7 @@ - (void)start {
   }
 
   if (self.delegate || self.luminanceLayer || self.binaryLayer) {
-    [self output];
+    (void)[self output];
   }
 
   if (!self.session.running) {

diff --git a/ZXingObjC/pdf417/ZXPDF417Reader.m b/ZXingObjC/pdf417/ZXPDF417Reader.m
@@ -100,14 +100,14 @@ - (int)maxWidth:(ZXResultPoint *)p1 p2:(ZXResultPoint *)p2 {
   if (!p1 || !p2 || (id)p1 == [NSNull null] || p2 == (id)[NSNull null]) {
     return 0;
   }
-  return abs(p1.x - p2.x);
+  return fabsf(p1.x - p2.x);
 }
 
 - (int)minWidth:(ZXResultPoint *)p1 p2:(ZXResultPoint *)p2 {
   if (!p1 || !p2 || (id)p1 == [NSNull null] || p2 == (id)[NSNull null]) {
     return INT_MAX;
   }
-  return abs(p1.x - p2.x);
+  return fabsf(p1.x - p2.x);
 }
 
 - (int)maxCodewordWidth:(NSArray *)p {