fix Xcode8 warning

ibireme · ibireme · commit a47f7728a0f3 · 2016-09-13T17:47:26.000+08:00
diff --git a/Framework/YYText.xcodeproj/project.pbxproj b/Framework/YYText.xcodeproj/project.pbxproj
@@ -315,7 +315,7 @@
 		D946DFD91BD2104E006B77F7 /* Project object */ = {
 			isa = PBXProject;
 			attributes = {
-				LastUpgradeCheck = 0700;
+				LastUpgradeCheck = 0800;
 				ORGANIZATIONNAME = ibireme;
 				TargetAttributes = {
 					D946DFE11BD2104E006B77F7 = {
@@ -398,8 +398,10 @@
 				CLANG_WARN_DIRECT_OBJC_ISA_USAGE = YES_ERROR;
 				CLANG_WARN_EMPTY_BODY = YES;
 				CLANG_WARN_ENUM_CONVERSION = YES;
+				CLANG_WARN_INFINITE_RECURSION = YES;
 				CLANG_WARN_INT_CONVERSION = YES;
 				CLANG_WARN_OBJC_ROOT_CLASS = YES_ERROR;
+				CLANG_WARN_SUSPICIOUS_MOVE = YES;
 				CLANG_WARN_UNREACHABLE_CODE = YES;
 				CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
 				"CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer";
@@ -422,7 +424,7 @@
 				GCC_WARN_UNINITIALIZED_AUTOS = YES_AGGRESSIVE;
 				GCC_WARN_UNUSED_FUNCTION = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;
-				IPHONEOS_DEPLOYMENT_TARGET = 9.0;
+				IPHONEOS_DEPLOYMENT_TARGET = 8.0;
 				MTL_ENABLE_DEBUG_INFO = YES;
 				ONLY_ACTIVE_ARCH = YES;
 				SDKROOT = iphoneos;
@@ -445,8 +447,10 @@
 				CLANG_WARN_DIRECT_OBJC_ISA_USAGE = YES_ERROR;
 				CLANG_WARN_EMPTY_BODY = YES;
 				CLANG_WARN_ENUM_CONVERSION = YES;
+				CLANG_WARN_INFINITE_RECURSION = YES;
 				CLANG_WARN_INT_CONVERSION = YES;
 				CLANG_WARN_OBJC_ROOT_CLASS = YES_ERROR;
+				CLANG_WARN_SUSPICIOUS_MOVE = YES;
 				CLANG_WARN_UNREACHABLE_CODE = YES;
 				CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
 				"CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer";
@@ -463,7 +467,7 @@
 				GCC_WARN_UNINITIALIZED_AUTOS = YES_AGGRESSIVE;
 				GCC_WARN_UNUSED_FUNCTION = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;
-				IPHONEOS_DEPLOYMENT_TARGET = 9.0;
+				IPHONEOS_DEPLOYMENT_TARGET = 8.0;
 				MTL_ENABLE_DEBUG_INFO = NO;
 				SDKROOT = iphoneos;
 				TARGETED_DEVICE_FAMILY = "1,2";
diff --git a/Framework/YYText.xcodeproj/xcshareddata/xcschemes/YYText.xcscheme b/Framework/YYText.xcodeproj/xcshareddata/xcschemes/YYText.xcscheme
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Scheme
-   LastUpgradeVersion = "0700"
+   LastUpgradeVersion = "0800"
    version = "1.3">
    <BuildAction
       parallelizeBuildables = "YES"
diff --git a/YYText/Component/YYTextLayout.m b/YYText/Component/YYTextLayout.m
@@ -111,7 +111,6 @@ + (instancetype)containerWithSize:(CGSize)size insets:(UIEdgeInsets)insets {
 }
 
 + (instancetype)containerWithPath:(UIBezierPath *)path {
-    if (!path) return nil;
     YYTextContainer *one = [self new];
     one.path = path;
     return one;
diff --git a/YYText/Component/YYTextLine.h b/YYText/Component/YYTextLine.h
@@ -53,9 +53,9 @@ NS_ASSUME_NONNULL_BEGIN
 @property (nonatomic, readonly) CGFloat lineWidth;  ///< line width
 @property (nonatomic, readonly) CGFloat trailingWhitespaceWidth;
 
-@property (nonatomic, readonly) NSArray<YYTextAttachment *> *attachments; ///< YYTextAttachment
-@property (nonatomic, readonly) NSArray<NSValue *> *attachmentRanges;     ///< NSRange(NSValue)
-@property (nonatomic, readonly) NSArray<NSValue *> *attachmentRects;      ///< CGRect(NSValue)
+@property (nullable, nonatomic, readonly) NSArray<YYTextAttachment *> *attachments; ///< YYTextAttachment
+@property (nullable, nonatomic, readonly) NSArray<NSValue *> *attachmentRanges;     ///< NSRange(NSValue)
+@property (nullable, nonatomic, readonly) NSArray<NSValue *> *attachmentRects;      ///< CGRect(NSValue)
 
 @end
 
diff --git a/YYText/Component/YYTextLine.m b/YYText/Component/YYTextLine.m
@@ -30,7 +30,7 @@ - (void)dealloc {
     if (_CTLine) CFRelease(_CTLine);
 }
 
-- (void)setCTLine:(CTLineRef)CTLine {
+- (void)setCTLine:(_Nonnull CTLineRef)CTLine {
     if (_CTLine != CTLine) {
         if (CTLine) CFRetain(CTLine);
         if (_CTLine) CFRelease(_CTLine);
diff --git a/YYText/Utility/NSParagraphStyle+YYText.m b/YYText/Utility/NSParagraphStyle+YYText.m
@@ -25,10 +25,13 @@ + (NSParagraphStyle *)yy_styleWithCTStyle:(CTParagraphStyleRef)CTStyle {
     
     NSMutableParagraphStyle *style = [[NSParagraphStyle defaultParagraphStyle] mutableCopy];
     
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
     CGFloat lineSpacing;
     if (CTParagraphStyleGetValueForSpecifier(CTStyle, kCTParagraphStyleSpecifierLineSpacing, sizeof(CGFloat), &lineSpacing)) {
         style.lineSpacing = lineSpacing;
     }
+#pragma clang diagnostic pop
     
     CGFloat paragraphSpacing;
     if (CTParagraphStyleGetValueForSpecifier(CTStyle, kCTParagraphStyleSpecifierParagraphSpacing, sizeof(CGFloat), &paragraphSpacing)) {
@@ -117,11 +120,14 @@ - (CTParagraphStyleRef)yy_CTStyle CF_RETURNS_RETAINED {
     CTParagraphStyleSetting set[kCTParagraphStyleSpecifierCount] = { 0 };
     int count = 0;
     
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
     CGFloat lineSpacing = self.lineSpacing;
     set[count].spec = kCTParagraphStyleSpecifierLineSpacing;
     set[count].valueSize = sizeof(CGFloat);
     set[count].value = &lineSpacing;
     count++;
+#pragma clang diagnostic pop
     
     CGFloat paragraphSpacing = self.paragraphSpacing;
     set[count].spec = kCTParagraphStyleSpecifierParagraphSpacing;
diff --git a/YYText/Utility/YYTextAsyncLayer.m b/YYText/Utility/YYTextAsyncLayer.m
@@ -117,7 +117,7 @@ - (void)display {
 #pragma mark - Private
 
 - (void)_displayAsync:(BOOL)async {
-    __strong id<YYTextAsyncLayerDelegate> delegate = self.delegate;
+    __strong id<YYTextAsyncLayerDelegate> delegate = (id)self.delegate;
     YYTextAsyncLayerDisplayTask *task = [delegate newAsyncDisplayTask];
     if (!task.display) {
         if (task.willDisplay) task.willDisplay(self);
diff --git a/YYText/YYLabel.m b/YYText/YYLabel.m
@@ -1156,7 +1156,7 @@ - (YYTextAsyncLayerDisplayTask *)newAsyncDisplayTask {
         }
         [layer removeAnimationForKey:@"contents"];
         
-        YYLabel *view = layer.delegate;
+        __strong YYLabel *view = (YYLabel *)layer.delegate;
         if (!view) return;
         if (view->_state.layoutNeedUpdate && layoutUpdated) {
             view->_innerLayout = layout;
diff --git a/YYText/YYTextView.m b/YYText/YYTextView.m
@@ -1073,7 +1073,7 @@ - (YYTextRange *)_getClosestTokenRangeAtPosition:(YYTextPosition *)position {
         }
     }
     
-    if (range.asRange.length == 0) {
+    if (!range || range.asRange.length == 0) {
         range = [_innerLayout textRangeByExtendingPosition:position inDirection:UITextLayoutDirectionRight offset:1];
         range = [self _correctedTextRange:range];
         if (range.asRange.length == 0) {

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,6 @@ + (instancetype)containerWithSize:(CGSize)size insets:(UIEdgeInsets)insets {`
`111`	`111`	`}`
`112`	`112`
`113`	`113`	`+ (instancetype)containerWithPath:(UIBezierPath *)path {`
`114`		`- if (!path) return nil;`
`115`	`114`	`YYTextContainer *one = [self new];`
`116`	`115`	`one.path = path;`
`117`	`116`	`return one;`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ - (void)dealloc {`
`30`	`30`	`if (_CTLine) CFRelease(_CTLine);`
`31`	`31`	`}`
`32`	`32`
`33`		`-- (void)setCTLine:(CTLineRef)CTLine {`
	`33`	`+- (void)setCTLine:(_Nonnull CTLineRef)CTLine {`
`34`	`34`	`if (_CTLine != CTLine) {`
`35`	`35`	`if (CTLine) CFRetain(CTLine);`
`36`	`36`	`if (_CTLine) CFRelease(_CTLine);`
Original file line number	Diff line number	Diff line change
`@@ -1156,7 +1156,7 @@ - (YYTextAsyncLayerDisplayTask *)newAsyncDisplayTask {`
`1156`	`1156`	`}`
`1157`	`1157`	`[layer removeAnimationForKey:@"contents"];`
`1158`	`1158`
`1159`		`- YYLabel *view = layer.delegate;`
	`1159`	`+ __strong YYLabel view = (YYLabel )layer.delegate;`
`1160`	`1160`	`if (!view) return;`
`1161`	`1161`	`if (view->_state.layoutNeedUpdate && layoutUpdated) {`
`1162`	`1162`	`view->_innerLayout = layout;`
Original file line number	Diff line number	Diff line change
`@@ -1073,7 +1073,7 @@ - (YYTextRange )_getClosestTokenRangeAtPosition:(YYTextPosition )position {`
`1073`	`1073`	`}`
`1074`	`1074`	`}`
`1075`	`1075`
`1076`		`- if (range.asRange.length == 0) {`
	`1076`	`+ if (!range \|\| range.asRange.length == 0) {`
`1077`	`1077`	`range = [_innerLayout textRangeByExtendingPosition:position inDirection:UITextLayoutDirectionRight offset:1];`
`1078`	`1078`	`range = [self _correctedTextRange:range];`
`1079`	`1079`	`if (range.asRange.length == 0) {`