Added warning flag -Wmissing-method-return-type and fixed resultant w…

…arnings: Method has no return type specified.
KosmicTask · Nov 26, 2012 · 0435a9e · 0435a9e
1 parent e09a74d
commit 0435a9e
Show file tree

Hide file tree

Showing 83 changed files with 320 additions and 311 deletions.
diff --git a/FScript.xcodeproj/project.pbxproj b/FScript.xcodeproj/project.pbxproj
@@ -3657,7 +3657,6 @@
 				PRODUCT_NAME = FScript;
 				SECTORDER_FLAGS = "";
 				SKIP_INSTALL = YES;
-				WARNING_CFLAGS = "-Wmost";
 				WRAPPER_EXTENSION = framework;
 				ZERO_LINK = NO;
 			};
@@ -3694,7 +3693,6 @@
 				PRODUCT_NAME = FScript;
 				SECTORDER_FLAGS = "";
 				SKIP_INSTALL = YES;
-				WARNING_CFLAGS = "-Wmost";
 				WRAPPER_EXTENSION = framework;
 				ZERO_LINK = NO;
 			};
@@ -3870,7 +3868,7 @@
 			buildSettings = {
 				ALWAYS_SEARCH_USER_PATHS = NO;
 				ARCHS = "$(ARCHS_STANDARD_32_64_BIT)";
-				GCC_C_LANGUAGE_STANDARD = gnu99;
+				GCC_C_LANGUAGE_STANDARD = c99;
 				GCC_ENABLE_CPP_EXCEPTIONS = NO;
 				GCC_ENABLE_CPP_RTTI = NO;
 				GCC_ENABLE_OBJC_EXCEPTIONS = YES;
@@ -3885,7 +3883,9 @@
 				GCC_WARN_EFFECTIVE_CPLUSPLUS_VIOLATIONS = YES;
 				GCC_WARN_INITIALIZER_NOT_FULLY_BRACKETED = YES;
 				GCC_WARN_MISSING_PARENTHESES = NO;
+				GCC_WARN_SHADOW = NO;
 				GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = YES;
+				GCC_WARN_UNDECLARED_SELECTOR = NO;
 				GCC_WARN_UNUSED_LABEL = YES;
 				GCC_WARN_UNUSED_PARAMETER = NO;
 				GCC_WARN_UNUSED_VALUE = YES;
@@ -3894,7 +3894,10 @@
 				MACOSX_DEPLOYMENT_TARGET = 10.6;
 				OTHER_LDFLAGS = "";
 				SDKROOT = macosx;
-				WARNING_CFLAGS = "-Wmost";
+				WARNING_CFLAGS = (
+					"-Wmost",
+					"-Wmissing-method-return-type",
+				);
 				ZERO_LINK = NO;
 			};
 			name = Debug;
@@ -3904,7 +3907,7 @@
 			buildSettings = {
 				ALWAYS_SEARCH_USER_PATHS = NO;
 				ARCHS = "$(ARCHS_STANDARD_32_64_BIT)";
-				GCC_C_LANGUAGE_STANDARD = gnu99;
+				GCC_C_LANGUAGE_STANDARD = c99;
 				GCC_ENABLE_CPP_EXCEPTIONS = NO;
 				GCC_ENABLE_CPP_RTTI = NO;
 				GCC_ENABLE_OBJC_EXCEPTIONS = YES;
@@ -3913,11 +3916,17 @@
 				GCC_MODEL_TUNING = "";
 				GCC_OPTIMIZATION_LEVEL = s;
 				GCC_VERSION = "";
+				GCC_WARN_SHADOW = NO;
+				GCC_WARN_UNDECLARED_SELECTOR = NO;
 				IBC_FLATTEN_NIBS = NO;
 				INSTALL_OWNER = "$(inherited)";
 				MACOSX_DEPLOYMENT_TARGET = 10.6;
 				OTHER_LDFLAGS = "";
 				SDKROOT = macosx;
+				WARNING_CFLAGS = (
+					"-Wmost",
+					"-Wmissing-method-return-type",
+				);
 			};
 			name = Release;
 		};

diff --git a/FScriptFramework/Array.m b/FScriptFramework/Array.m
@@ -163,9 +163,9 @@ - (BOOL)containsObject:(id)anObject
   assert(0);
 }
 
-- copy  {  assert(0);  }
+- (id)copy  {  assert(0);  }
 
-- copyWithZone:(NSZone *)zone  {   assert(0);   }
+- (id)copyWithZone:(NSZone *)zone  {   assert(0);   }
 
 - (NSUInteger)count 
 {
@@ -252,32 +252,32 @@ - (NSUInteger)indexOfObjectIdenticalTo:(id)anObject inRange:(NSRange)range
   assert(0);
 }
 
-- init
+- (id)init
 {
   assert(0);
 }   
 
-- initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step
+- (id)initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step
 {
   assert(0);
 }           
 
-- initFilledWith:(id)elem count:(NSUInteger)nb
+- (id)initFilledWith:(id)elem count:(NSUInteger)nb
 {
   assert(0);
 } 
 
-- initWithCapacity:(NSUInteger)aNumItems
+- (id)initWithCapacity:(NSUInteger)aNumItems
 {
   assert(0);
 }
 
-- initWithObject:(id)object
+- (id)initWithObject:(id)object
 {
   assert(0);
 }
 
-- initWithObjects:(id *)objects count:(NSUInteger)nb
+- (id)initWithObjects:(id *)objects count:(NSUInteger)nb
 {
   assert(0);
 }
@@ -302,12 +302,12 @@ - (BOOL) isEqualToArray:(NSArray *)anArray
   assert(0);
 }
 
-- mutableCopyWithZone:(NSZone *)zone 
+- (id)mutableCopyWithZone:(NSZone *)zone 
 { 
   assert(0);
 }
 
-- objectAtIndex:(NSUInteger)index
+- (id)objectAtIndex:(NSUInteger)index
 {
   assert(0);
 }   
@@ -373,7 +373,7 @@ - (enum ArrayRepType)type  {   assert(0);   } // declared in ArrayPrivate
 
 -(NSUInteger) _ul_count {   assert(0);   }
 
-- _ul_objectAtIndex:(NSUInteger)index
+- (id)_ul_objectAtIndex:(NSUInteger)index
 { 
   assert(0);
 }

diff --git a/FScriptFramework/ArrayPrivate.h b/FScriptFramework/ArrayPrivate.h
@@ -15,8 +15,8 @@
 - (id)arrayRep;
 - (void)becomeArrayOfId;
 - (void *)dataPtr;
-- initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step;
-- initFilledWith:(id)elem count:(NSUInteger)nb;
+- (id)initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step;
+- (id)initFilledWith:(id)elem count:(NSUInteger)nb;
 - (FSArray *)initWithRep:(id)theRep;
 - (FSArray *)initWithRepNoRetain:(id)theRep;
 - (enum ArrayRepType)type;

diff --git a/FScriptFramework/ArrayRep.h b/FScriptFramework/ArrayRep.h
@@ -18,8 +18,8 @@
 - (NSString *)descriptionLimited:(NSUInteger)nbElem;
 - (FSArray *) distinctId; 
 - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)identical;
-- indexWithArray:(FSArray *)index;
-- (id)operator_backslash:(FSBlock*)bl; // precond: ![bl isProxy] && count != 0 
+- (id)indexWithArray:(FSArray *)index;
+- (id)operator_backslash:(FSBlock*)bl; // precond: ![bl isProxy] && count != 0
 - (void)removeLastElem;
 - (void)removeElemAtIndex:(NSUInteger)index;
 - (FSArray *)replicateWithArray:(FSArray *)operand;

diff --git a/FScriptFramework/ArrayRepBoolean.h b/FScriptFramework/ArrayRepBoolean.h
@@ -17,12 +17,12 @@
 - (void)addBoolean:(char)aBoolean;
 - (char *)booleansPtr;
 - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)identical;
-- init;
-- initFilledWithBoolean:(char)elem count:(NSUInteger)nb; // contract: a return value of nil means not enough memory
-- initWithCapacity:(NSUInteger)aNumItems; // contract: a return value of nil means not enough memory
-- initWithBooleans:(char *)elems count:(NSUInteger)nb;
-- initWithBooleansNoCopy:(char *)tab count:(NSUInteger)nb;
-- copyWithZone:(NSZone *)zone;
+- (id)init;
+- (id)initFilledWithBoolean:(char)elem count:(NSUInteger)nb; // contract: a return value of nil means not enough memory
+- (id)initWithCapacity:(NSUInteger)aNumItems; // contract: a return value of nil means not enough memory
+- (id)initWithBooleans:(char *)elems count:(NSUInteger)nb;
+- (id)initWithBooleansNoCopy:(char *)tab count:(NSUInteger)nb;
+- (id)copyWithZone:(NSZone *)zone;
 - (void)replaceBooleanAtIndex:(NSUInteger)index withBoolean:(char)aBoolean;
 - (enum ArrayRepType)repType;
 

diff --git a/FScriptFramework/ArrayRepBoolean.m b/FScriptFramework/ArrayRepBoolean.m
@@ -184,7 +184,7 @@ - (ArrayRepId *) asArrayRepId
 
 - (char *)booleansPtr {return t;}
 
-- copyWithZone:(NSZone *)zone
+- (id)copyWithZone:(NSZone *)zone
 {
   return [[ArrayRepBoolean allocWithZone:zone] initWithBooleans:t count:count];  
 }
@@ -220,9 +220,9 @@ - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)
   }    
 }
 
-- init { return [self initWithCapacity:0]; }
+- (id)init { return [self initWithCapacity:0]; }
 
-- initFilledWithBoolean:(char)elem count:(NSUInteger)nb // contract: a return value of nil means not enough memory
+- (id)initFilledWithBoolean:(char)elem count:(NSUInteger)nb // contract: a return value of nil means not enough memory
 {
   if (self = [self initWithCapacity:nb])
   {    
@@ -232,7 +232,7 @@ - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)
   return nil;
 }
 
-- initWithCapacity:(NSUInteger)aNumItems // contract: a return value of nil means not enough memory  
+- (id)initWithCapacity:(NSUInteger)aNumItems // contract: a return value of nil means not enough memory  
 {
   if ((self = [super init]))
   {
@@ -250,7 +250,7 @@ - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)
   return nil;    
 }
 
-- initWithBooleans:(char *)elems count:(NSUInteger)nb
+- (id)initWithBooleans:(char *)elems count:(NSUInteger)nb
 {  
   if (self = [self initWithCapacity:nb])
   {
@@ -261,7 +261,7 @@ - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)
   return nil;
 }
 
-- initWithBooleansNoCopy:(char *)tab count:(NSUInteger)nb
+- (id)initWithBooleansNoCopy:(char *)tab count:(NSUInteger)nb
 {
   if ((self = [super init]))
   {

diff --git a/FScriptFramework/ArrayRepDouble.h b/FScriptFramework/ArrayRepDouble.h
@@ -14,20 +14,20 @@
 }
 
 
-+ arrayRepDoubleWithCapacity:(NSUInteger)aNumItems;
++ (id)arrayRepDoubleWithCapacity:(NSUInteger)aNumItems;
 
 - (void)addDouble:(double)aDouble;
 - (void)addDoublesFromFSArray:(FSArray *)otherArray;
 - (NSString *)descriptionLimited:(NSUInteger)nbElem;
 - (NSUInteger)indexOfObject:(id)anObject inRange:(NSRange)range identical:(BOOL)identical;
-- init;
-- initFilledWithDouble:(double)elem count:(NSUInteger)nb; // contract: a return value of nil means not enough memory
-- initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step; // contract: a return value of nil means not enough memory
-- initWithCapacity:(NSUInteger)aNumItems; // contract: a return value of nil means not enough memory
-- initWithDoubles:(double *)elems count:(NSUInteger)nb;
-- initWithDoublesNoCopy:(double *)tab count:(NSUInteger)nb;
+- (id)init;
+- (id)initFilledWithDouble:(double)elem count:(NSUInteger)nb; // contract: a return value of nil means not enough memory
+- (id)initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step; // contract: a return value of nil means not enough memory
+- (id)initWithCapacity:(NSUInteger)aNumItems; // contract: a return value of nil means not enough memory
+- (id)initWithDoubles:(double *)elems count:(NSUInteger)nb;
+- (id)initWithDoublesNoCopy:(double *)tab count:(NSUInteger)nb;
 - (void)insertDouble:(double)aDouble atIndex:(NSUInteger)index;
-- copyWithZone:(NSZone *)zone;
+- (id)copyWithZone:(NSZone *)zone;
 - (double *)doublesPtr;
 - (void)replaceDoubleAtIndex:(NSUInteger)index withDouble:(double)aDouble;
 - (enum ArrayRepType)repType;

diff --git a/FScriptFramework/ArrayRepDouble.m b/FScriptFramework/ArrayRepDouble.m
@@ -894,7 +894,7 @@ - (FSArray *)simpleLoop_truncated
 /////////////////////////////// OTHER METHODS //////////////////////////////////
 ////////////////////////////////////////////////////////////////////////////////
 
-+ arrayRepDoubleWithCapacity:(NSUInteger)aNumItems
++ (id)arrayRepDoubleWithCapacity:(NSUInteger)aNumItems
 {
   return [[[self alloc] initWithCapacity:aNumItems] autorelease];
 }
@@ -938,7 +938,7 @@ - (ArrayRepId *) asArrayRepId
   return r;
 }
 
-- copyWithZone:(NSZone *)zone
+- (id)copyWithZone:(NSZone *)zone
 {
   return [[ArrayRepDouble allocWithZone:zone] initWithDoubles:t count:count];  
 }
@@ -987,7 +987,7 @@ - (NSString *)descriptionLimited:(NSUInteger)nbElem
   return str; 
 }
 
-- indexWithArray:(FSArray *)index
+- (id)indexWithArray:(FSArray *)index
 {
   assert(![index isProxy]);
 
@@ -1121,9 +1121,9 @@ - (NSString *)descriptionLimited:(NSUInteger)nbElem
   return nil; //W
 }
 
-- init { return [self initWithCapacity:0]; }
+- (id)init { return [self initWithCapacity:0]; }
 
-- initFilledWithDouble:(double)elem count:(NSUInteger)nb // contract: a return value of nil means not enough memory
+- (id)initFilledWithDouble:(double)elem count:(NSUInteger)nb // contract: a return value of nil means not enough memory
 {
   if (self = [self initWithCapacity:nb])
   {    
@@ -1133,7 +1133,7 @@ - (NSString *)descriptionLimited:(NSUInteger)nbElem
   return nil;
 }
 
-- initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step // contract: a return value of nil means not enough memory
+- (id)initFrom:(NSUInteger)from to:(NSUInteger)to step:(NSUInteger)step // contract: a return value of nil means not enough memory
 {
   if (to < from) return [self init];
 
@@ -1153,7 +1153,7 @@ - (NSString *)descriptionLimited:(NSUInteger)nbElem
   return nil;
 }          
 
-- initWithCapacity:(NSUInteger)aNumItems // contract: a return value of nil means not enough memory
+- (id)initWithCapacity:(NSUInteger)aNumItems // contract: a return value of nil means not enough memory
 {
   if (self = [super init])
   {
@@ -1171,7 +1171,7 @@ - (NSString *)descriptionLimited:(NSUInteger)nbElem
   return nil;    
 }
 
-- initWithDoubles:(double *)elems count:(NSUInteger)nb
+- (id)initWithDoubles:(double *)elems count:(NSUInteger)nb
 {  
   if ((self = [self initWithCapacity:nb]))
   {
@@ -1182,7 +1182,7 @@ - (NSString *)descriptionLimited:(NSUInteger)nbElem
   return nil;
 }
 
-- initWithDoublesNoCopy:(double *)tab count:(NSUInteger)nb
+- (id)initWithDoublesNoCopy:(double *)tab count:(NSUInteger)nb
 {
   if ((self = [super init]))
   {

diff --git a/FScriptFramework/ArrayRepEmpty.h b/FScriptFramework/ArrayRepEmpty.h
@@ -22,7 +22,7 @@
 
 - (ArrayRepDouble *)  asArrayRepDouble;
 - (ArrayRepBoolean *) asArrayRepBoolean;
-- init;
-- initWithCapacity:(NSUInteger)aNumItems;
+- (id)init;
+- (id)initWithCapacity:(NSUInteger)aNumItems;
 
 @end
diff --git a/FScriptFramework/ArrayRepEmpty.m b/FScriptFramework/ArrayRepEmpty.m
@@ -28,7 +28,7 @@ - (FSArray *)sort    {return [FSArray array];}
 /////////////////////////////// OTHER METHODS //////////////////////////////////
 ////////////////////////////////////////////////////////////////////////////////
 
-+ arrayRepEmptyWithCapacity:(NSUInteger)aNumItems
++ (id)arrayRepEmptyWithCapacity:(NSUInteger)aNumItems
 {
   return [[[self alloc] initWithCapacity:aNumItems] autorelease];
 }
@@ -47,7 +47,7 @@ - (ArrayRepDouble *) asArrayRepDouble   { return [[[ArrayRepDouble alloc] initWi
 
 - (ArrayRepId *)     asArrayRepId       { return [[[ArrayRepId     alloc] initWithCapacity:capacity] autorelease];  }
 
-- copyWithZone:(NSZone *)zone
+- (id)copyWithZone:(NSZone *)zone
 {
   return [[ArrayRepEmpty allocWithZone:zone] initWithCapacity:capacity];  
 }
@@ -56,11 +56,11 @@ - (NSUInteger)count {return 0;}
 
 - (NSString *)descriptionLimited:(NSUInteger)nbElem { return @"{}" ; }
 
-- indexWithArray:(FSArray *)index {return [FSArray array];}
+- (id)indexWithArray:(FSArray *)index {return [FSArray array];}
 
-- init { return [self initWithCapacity:0]; }
+- (id)init { return [self initWithCapacity:0]; }
 
-- initWithCapacity:(NSUInteger)aNumItems
+- (id)initWithCapacity:(NSUInteger)aNumItems
 {
   if ((self = [super init]))
   {

diff --git a/FScriptFramework/ArrayRepFetchRequest.h b/FScriptFramework/ArrayRepFetchRequest.h
@@ -14,9 +14,9 @@
 + (ArrayRepFetchRequest *) arrayRepFetchRequestWithFetchRequest:(NSFetchRequest *)theFetchRequest objectContext:(NSManagedObjectContext *)theObjectContext;
 
 - (ArrayRepId *) asArrayRepId; 
-- copyWithZone:(NSZone *)zone;
+- (id)copyWithZone:(NSZone *)zone;
 - (void)dealloc;
-- initWithFetchRequest:(NSFetchRequest *)theFetchRequest objectContext:(NSManagedObjectContext *)theObjectContext;
+- (id)initWithFetchRequest:(NSFetchRequest *)theFetchRequest objectContext:(NSManagedObjectContext *)theObjectContext;
 - (enum ArrayRepType) repType;
 
 @end