Skip to content

Commit 2382c58

Browse files
authored
Merge pull request SDWebImage#3484 from dreampiggy/garden/objc_bool_yes_no
Code garden to change Objc bool from true/false to YES/NO
2 parents 6b25d98 + f2d7610 commit 2382c58

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

SDWebImage/Core/SDAnimatedImagePlayer.m

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -249,9 +249,9 @@ - (void)displayDidRefresh:(SDDisplayLink *)displayLink {
249249
} else if (self.playbackMode == SDAnimatedImagePlaybackModeBounce ||
250250
self.playbackMode == SDAnimatedImagePlaybackModeReversedBounce) {
251251
if (currentFrameIndex == 0) {
252-
self.shouldReverse = false;
252+
self.shouldReverse = NO;
253253
} else if (currentFrameIndex == totalFrameCount - 1) {
254-
self.shouldReverse = true;
254+
self.shouldReverse = YES;
255255
}
256256
nextFrameIndex = self.shouldReverse ? (currentFrameIndex - 1) : (currentFrameIndex + 1);
257257
nextFrameIndex %= totalFrameCount;

SDWebImage/Core/SDWebImageDownloader.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -518,7 +518,7 @@ - (void)URLSession:(NSURLSession *)session task:(NSURLSessionTask *)task didComp
518518
if (dataOperation) {
519519
@synchronized (dataOperation) {
520520
// Mark the downloader operation `isCompleted = YES`, no longer re-use this operation when new request comes in
521-
SDWebImageDownloaderOperationSetCompleted(dataOperation, true);
521+
SDWebImageDownloaderOperationSetCompleted(dataOperation, YES);
522522
}
523523
}
524524
if ([dataOperation respondsToSelector:@selector(URLSession:task:didCompleteWithError:)]) {

Tests/Tests/SDAnimatedImageTest.m

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -680,7 +680,7 @@ - (void)test34AnimatedImagePlaybackModeBounce {
680680
player.playbackMode = SDAnimatedImagePlaybackModeBounce;
681681

682682
__block NSInteger i = 0;
683-
__block BOOL flag = false;
683+
__block BOOL flag = NO;
684684
__block NSUInteger cnt = 0;
685685
__weak typeof(imageView) wimageView = imageView;
686686
[player setAnimationFrameHandler:^(NSUInteger index, UIImage * _Nonnull frame) {
@@ -689,10 +689,10 @@ - (void)test34AnimatedImagePlaybackModeBounce {
689689

690690
if (index >= player.totalFrameCount - 1) {
691691
cnt++;
692-
flag = true;
692+
flag = YES;
693693
} else if (cnt != 0 && index == 0) {
694694
cnt++;
695-
flag = false;
695+
flag = NO;
696696
}
697697

698698
if (!flag) {
@@ -745,7 +745,7 @@ - (void)test35AnimatedImagePlaybackModeReversedBounce {
745745
flag = false;
746746
} else if (index == 0) {
747747
cnt++;
748-
flag = true;
748+
flag = YES;
749749
}
750750

751751
if (flag) {

Tests/Tests/SDImageCacheTests.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ - (void)test15CancelQueryShouldCallbackOnceInSync {
233233
[SDImageCache.sharedImageCache removeImageFromDiskForKey:key];
234234
__block BOOL callced = NO;
235235
SDImageCacheToken *token = [SDImageCache.sharedImageCache queryCacheOperationForKey:key done:^(UIImage * _Nullable image, NSData * _Nullable data, SDImageCacheType cacheType) {
236-
callced = true;
236+
callced = YES;
237237
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, 0), dispatch_get_main_queue(), ^{
238238
[expectation fulfill]; // callback once fulfill once
239239
});

0 commit comments

Comments
 (0)