Skip to content

Commit d85e97d

Browse files
committed
enabled and fixed additional warnings
1 parent 54a8a84 commit d85e97d

File tree

4 files changed

+38
-4
lines changed

4 files changed

+38
-4
lines changed

THPinViewController/THPinView.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ - (instancetype)init {
3333
return nil;
3434
}
3535

36-
- (instancetype)initWithDelegate:(id<THPinViewDelegate>)delegate;
36+
- (instancetype)initWithDelegate:(id<THPinViewDelegate>)delegate
3737
{
3838
self = [super init];
3939
if (self)

THPinViewController/THPinViewController.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ - (NSUInteger)pinLengthForPinView:(THPinView *)pinView
113113
{
114114
NSUInteger pinLength = [self.delegate pinLengthForPinViewController:self];
115115
NSAssert(pinLength > 0, @"PIN length must be greater than 0");
116-
return MAX(pinLength, 1);
116+
return MAX(pinLength, (NSUInteger)1);
117117
}
118118

119119
- (BOOL)pinView:(THPinView *)pinView isPinValid:(NSString *)pin

THPinViewControllerExample.xcodeproj/project.pbxproj

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -373,16 +373,21 @@
373373
isa = XCBuildConfiguration;
374374
buildSettings = {
375375
ALWAYS_SEARCH_USER_PATHS = NO;
376+
CLANG_ANALYZER_SECURITY_FLOATLOOPCOUNTER = YES;
377+
CLANG_ANALYZER_SECURITY_INSECUREAPI_RAND = YES;
376378
CLANG_CXX_LANGUAGE_STANDARD = "gnu++0x";
377379
CLANG_CXX_LIBRARY = "libc++";
378380
CLANG_ENABLE_MODULES = YES;
379381
CLANG_ENABLE_OBJC_ARC = YES;
380382
CLANG_WARN_BOOL_CONVERSION = YES;
381383
CLANG_WARN_CONSTANT_CONVERSION = YES;
384+
CLANG_WARN_DEPRECATED_OBJC_IMPLEMENTATIONS = YES;
382385
CLANG_WARN_DIRECT_OBJC_ISA_USAGE = YES_ERROR;
383386
CLANG_WARN_EMPTY_BODY = YES;
384387
CLANG_WARN_ENUM_CONVERSION = YES;
388+
CLANG_WARN_IMPLICIT_SIGN_CONVERSION = YES;
385389
CLANG_WARN_INT_CONVERSION = YES;
390+
CLANG_WARN_OBJC_IMPLICIT_RETAIN_SELF = YES;
386391
CLANG_WARN_OBJC_ROOT_CLASS = YES_ERROR;
387392
CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
388393
"CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer";
@@ -395,47 +400,78 @@
395400
"$(inherited)",
396401
);
397402
GCC_SYMBOLS_PRIVATE_EXTERN = NO;
403+
GCC_TREAT_WARNINGS_AS_ERRORS = YES;
398404
GCC_WARN_64_TO_32_BIT_CONVERSION = YES;
405+
GCC_WARN_ABOUT_MISSING_FIELD_INITIALIZERS = YES;
406+
GCC_WARN_ABOUT_MISSING_NEWLINE = YES;
399407
GCC_WARN_ABOUT_RETURN_TYPE = YES_ERROR;
408+
GCC_WARN_INITIALIZER_NOT_FULLY_BRACKETED = YES;
409+
GCC_WARN_SHADOW = YES;
410+
GCC_WARN_SIGN_COMPARE = YES;
400411
GCC_WARN_UNDECLARED_SELECTOR = YES;
401412
GCC_WARN_UNINITIALIZED_AUTOS = YES_AGGRESSIVE;
402413
GCC_WARN_UNUSED_FUNCTION = YES;
414+
GCC_WARN_UNUSED_LABEL = YES;
403415
GCC_WARN_UNUSED_VARIABLE = YES;
404416
IPHONEOS_DEPLOYMENT_TARGET = 7.0;
405417
ONLY_ACTIVE_ARCH = YES;
418+
RUN_CLANG_STATIC_ANALYZER = YES;
406419
SDKROOT = iphoneos;
420+
WARNING_CFLAGS = (
421+
"-Wextra",
422+
"-Wno-unused-parameter",
423+
"-Wunreachable-code",
424+
);
407425
};
408426
name = Debug;
409427
};
410428
E26048CF19027D01006ACBC7 /* Release */ = {
411429
isa = XCBuildConfiguration;
412430
buildSettings = {
413431
ALWAYS_SEARCH_USER_PATHS = NO;
432+
CLANG_ANALYZER_SECURITY_FLOATLOOPCOUNTER = YES;
433+
CLANG_ANALYZER_SECURITY_INSECUREAPI_RAND = YES;
414434
CLANG_CXX_LANGUAGE_STANDARD = "gnu++0x";
415435
CLANG_CXX_LIBRARY = "libc++";
416436
CLANG_ENABLE_MODULES = YES;
417437
CLANG_ENABLE_OBJC_ARC = YES;
418438
CLANG_WARN_BOOL_CONVERSION = YES;
419439
CLANG_WARN_CONSTANT_CONVERSION = YES;
440+
CLANG_WARN_DEPRECATED_OBJC_IMPLEMENTATIONS = YES;
420441
CLANG_WARN_DIRECT_OBJC_ISA_USAGE = YES_ERROR;
421442
CLANG_WARN_EMPTY_BODY = YES;
422443
CLANG_WARN_ENUM_CONVERSION = YES;
444+
CLANG_WARN_IMPLICIT_SIGN_CONVERSION = YES;
423445
CLANG_WARN_INT_CONVERSION = YES;
446+
CLANG_WARN_OBJC_IMPLICIT_RETAIN_SELF = YES;
424447
CLANG_WARN_OBJC_ROOT_CLASS = YES_ERROR;
425448
CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
426449
"CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer";
427450
COPY_PHASE_STRIP = YES;
428451
ENABLE_NS_ASSERTIONS = NO;
429452
GCC_C_LANGUAGE_STANDARD = gnu99;
453+
GCC_TREAT_WARNINGS_AS_ERRORS = YES;
430454
GCC_WARN_64_TO_32_BIT_CONVERSION = YES;
455+
GCC_WARN_ABOUT_MISSING_FIELD_INITIALIZERS = YES;
456+
GCC_WARN_ABOUT_MISSING_NEWLINE = YES;
431457
GCC_WARN_ABOUT_RETURN_TYPE = YES_ERROR;
458+
GCC_WARN_INITIALIZER_NOT_FULLY_BRACKETED = YES;
459+
GCC_WARN_SHADOW = YES;
460+
GCC_WARN_SIGN_COMPARE = YES;
432461
GCC_WARN_UNDECLARED_SELECTOR = YES;
433462
GCC_WARN_UNINITIALIZED_AUTOS = YES_AGGRESSIVE;
434463
GCC_WARN_UNUSED_FUNCTION = YES;
464+
GCC_WARN_UNUSED_LABEL = YES;
435465
GCC_WARN_UNUSED_VARIABLE = YES;
436466
IPHONEOS_DEPLOYMENT_TARGET = 7.0;
467+
RUN_CLANG_STATIC_ANALYZER = YES;
437468
SDKROOT = iphoneos;
438469
VALIDATE_PRODUCT = YES;
470+
WARNING_CFLAGS = (
471+
"-Wextra",
472+
"-Wno-unused-parameter",
473+
"-Wunreachable-code",
474+
);
439475
};
440476
name = Release;
441477
};

THPinViewControllerExample/THPinViewControllerExample-Info.plist

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@
2424
<string>1.0</string>
2525
<key>LSRequiresIPhoneOS</key>
2626
<true/>
27-
<key>NSMainNibFile~ipad</key>
28-
<string>Main-iPad</string>
2927
<key>UIMainStoryboardFile</key>
3028
<string>Main</string>
3129
<key>UIRequiredDeviceCapabilities</key>

0 commit comments

Comments
 (0)