[Infer] Fix Infer errors/warnings (#1938)

* [Infer] Fix 11 Infer errors/warnings

* fix build error

AsyncDisplayKit/ASControlNode.mm

@@ -286,7 +286,9 @@ void _ASEnumerateControlEventsIncludedInMaskWithBlock(ASControlNodeEvent mask, v
       {
         // Create the dispatch table for this event.
         eventDispatchTable = [NSMapTable weakToStrongObjectsMapTable];
-        _controlEventDispatchTable[eventKey] = eventDispatchTable;
+        if (eventKey) {
+          [_controlEventDispatchTable setObject:eventDispatchTable forKey:eventKey];
+        }
       }
 
       // Have we seen this target before for this event?
@@ -442,9 +444,11 @@ id<NSCopying> _ASControlNodeEventKeyForControlEvent(ASControlNodeEvent controlEv
 
 void _ASEnumerateControlEventsIncludedInMaskWithBlock(ASControlNodeEvent mask, void (^block)(ASControlNodeEvent anEvent))
 {
+  if (block == nil) {
+    return;
+  }
   // Start with our first event (touch down) and work our way up to the last event (touch cancel)
-  for (ASControlNodeEvent thisEvent = ASControlNodeEventTouchDown; thisEvent <= ASControlNodeEventTouchCancel; thisEvent <<= 1)
+  for (ASControlNodeEvent thisEvent = ASControlNodeEventTouchDown; thisEvent <= ASControlNodeEventTouchCancel; thisEvent <<= 1){
-  {
     // If it's included in the mask, invoke the block.
     if ((mask & thisEvent) == thisEvent)
       block(thisEvent);

AsyncDisplayKit/ASRunLoopQueue.mm

@@ -59,6 +59,7 @@ static void runLoopSourceCallback(void *info) {
     // It is not guaranteed that the runloop will turn if it has no scheduled work, and this causes processing of
     // the queue to stop. Attaching a custom loop source to the run loop and signal it if new work needs to be done
     CFRunLoopSourceContext *runLoopSourceContext = (CFRunLoopSourceContext *)calloc(1, sizeof(CFRunLoopSourceContext));
+    if (runLoopSourceContext) {
       runLoopSourceContext->perform = runLoopSourceCallback;
 #if ASRunLoopQueueLoggingEnabled
       runLoopSourceContext->info = (__bridge void *)self;
@@ -66,6 +67,7 @@ static void runLoopSourceCallback(void *info) {
       _runLoopSource = CFRunLoopSourceCreate(NULL, 0, runLoopSourceContext);
       CFRunLoopAddSource(runloop, _runLoopSource, kCFRunLoopCommonModes);
       free(runLoopSourceContext);
+    }
 
 #if ASRunLoopQueueLoggingEnabled
     _runloopQueueLoggingTimer = [NSTimer timerWithTimeInterval:1.0 target:self selector:@selector(checkRunLoop) userInfo:nil repeats:YES];

AsyncDisplayKit/ASTableView.mm

@@ -606,6 +606,7 @@ static NSString * const kCellReuseIdentifier = @"_ASTableViewCell";
   cell.delegate = self;
 
   ASCellNode *node = [_dataController nodeAtIndexPath:indexPath];
+  if (node) {
     [_rangeController configureContentView:cell.contentView forCellNode:node];
 
     cell.node = node;
@@ -616,6 +617,7 @@ static NSString * const kCellReuseIdentifier = @"_ASTableViewCell";
     // This is actually a workaround for a bug we are seeing in some rare cases (selected background view
     // overlaps other cells if size of ASCellNode has changed.)
     cell.clipsToBounds = node.clipsToBounds;
+  }
 
   return cell;
 }

AsyncDisplayKit/ASVideoPlayerNode.mm

@@ -379,19 +379,19 @@ static void *ASVideoPlayerNodeContext = &ASVideoPlayerNodeContext;
       slider.maximumValue = 1.0;
 
       if (_delegateFlags.delegateScrubberMinimumTrackTintColor) {
-        slider.minimumTrackTintColor  = [_delegate videoPlayerNodeScrubberMinimumTrackTint:strongSelf];
+        slider.minimumTrackTintColor  = [strongSelf.delegate videoPlayerNodeScrubberMinimumTrackTint:strongSelf];
       }
 
       if (_delegateFlags.delegateScrubberMaximumTrackTintColor) {
-        slider.maximumTrackTintColor  = [_delegate videoPlayerNodeScrubberMaximumTrackTint:strongSelf];
+        slider.maximumTrackTintColor  = [strongSelf.delegate videoPlayerNodeScrubberMaximumTrackTint:strongSelf];
       }
 
       if (_delegateFlags.delegateScrubberThumbTintColor) {
-        slider.thumbTintColor  = [_delegate videoPlayerNodeScrubberThumbTint:strongSelf];
+        slider.thumbTintColor  = [strongSelf.delegate videoPlayerNodeScrubberThumbTint:strongSelf];
       }
 
       if (_delegateFlags.delegateScrubberThumbImage) {
-        UIImage *thumbImage = [_delegate videoPlayerNodeScrubberThumbImage:strongSelf];
+        UIImage *thumbImage = [strongSelf.delegate videoPlayerNodeScrubberThumbImage:strongSelf];
         [slider setThumbImage:thumbImage forState:UIControlStateNormal];
       }
 

AsyncDisplayKit/Details/ASDataController.mm

@@ -560,6 +560,10 @@ NSString * const ASDataControllerRowNodeKind = @"_ASDataControllerRowNodeKind";
     return;
   }
   
+  if (block == nil) {
+    return;
+  }
+  
   // If we have never performed a reload, there is no value in executing edit operations as the initial
   // reload will directly re-query the latest state of the datasource - so completely skip the block in this case.
   if (_batchUpdateCounter == 0) {

AsyncDisplayKit/Details/NSIndexSet+ASHelpers.m

@@ -68,9 +68,9 @@
   NSMutableString *result = [NSMutableString stringWithString:@"{ "];
   [self enumerateRangesUsingBlock:^(NSRange range, BOOL * _Nonnull stop) {
     if (range.length == 1) {
-      [result appendFormat:@"%zu ", range.location];
+      [result appendFormat:@"%zu ", (unsigned long)range.location];
     } else {
-      [result appendFormat:@"%zu-%zu ", range.location, NSMaxRange(range) - 1];
+      [result appendFormat:@"%zu-%lu ", (unsigned long)range.location, NSMaxRange(range) - 1];
     }
   }];
   [result appendString:@"}"];

AsyncDisplayKit/Private/ASInternalHelpers.mm

@@ -34,6 +34,9 @@ BOOL ASSubclassOverridesClassSelector(Class superclass, Class subclass, SEL sele
 
 void ASPerformBlockOnMainThread(void (^block)())
 {
+  if (block == nil){
+    return;
+  }
   if (ASDisplayNodeThreadIsMain()) {
     block();
   } else {
@@ -43,6 +46,9 @@ void ASPerformBlockOnMainThread(void (^block)())
 
 void ASPerformBlockOnBackgroundThread(void (^block)())
 {
+  if (block == nil){
+    return;
+  }
   if (ASDisplayNodeThreadIsMain()) {
     dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), block);
   } else {

AsyncDisplayKit/Private/ASMultidimensionalArrayUtils.mm

@@ -43,9 +43,11 @@ static void ASRecursivelyUpdateMultidimensionalArrayAtIndexPaths(NSMutableArray
       curIdx++;
     }
 
+    if (updateBlock){
       updateBlock(mutableArray, indexSet, curIdx);
     }
   }
+}
 
 static void ASRecursivelyFindIndexPathsForMultidimensionalArray(NSObject *obj, NSIndexPath *curIndexPath, NSMutableArray <NSIndexPath *>*res)
 {

AsyncDisplayKit/TextKit/ASTextKitContext.mm

@@ -75,7 +75,9 @@
                                                           NSTextContainer *))block
 {
   std::lock_guard<std::mutex> l(_textKitMutex);
+  if (block) {
     block(_layoutManager, _textStorage, _textContainer);
   }
+}
 
 @end

inferScript.sh

@@ -0,0 +1,8 @@
+#!/bin/sh
+
+if ! [ -x "$(command -v infer)" ]; then
+    echo "infer not found"
+    echo "Install infer with homebrew: brew install infer"
+else
+    infer --continue --reactive -- xcodebuild build -workspace AsyncDisplayKit.xcworkspace -scheme "AsyncDisplayKit-iOS" -configuration Debug -sdk iphonesimulator9.3
+fi