Changed nullable keyword to _Nullable and added a missing return

Author: Jerome Boursier

Source/IGListKit/IGListBindingSectionController.m

@@ -155,7 +155,7 @@ - (void)didUnhighlightItemAtIndex:(NSInteger)index {
 }
 
 #if !TARGET_OS_TV
-- (nullable UIContextMenuConfiguration *)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point {
+- (UIContextMenuConfiguration * _Nullable)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point {
     return [self.selectionDelegate sectionController:self contextMenuConfigurationForItemAtIndex:index point:point viewModel:self.viewModels[index]];
 }
 #endif

Source/IGListKit/IGListBindingSectionControllerSelectionDelegate.h

@@ -71,10 +71,10 @@ didUnhighlightItemAtIndex:(NSInteger)index
  
  @return An object that conforms to `UIContextMenuConfiguration`.
  */
-- (nullable UIContextMenuConfiguration *)sectionController:(IGListBindingSectionController *)sectionController
-                    contextMenuConfigurationForItemAtIndex:(NSInteger)index
-                                                     point:(CGPoint)point
-                                                 viewModel:(id)viewModel API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos);
+- (UIContextMenuConfiguration * _Nullable)sectionController:(IGListBindingSectionController *)sectionController
+                     contextMenuConfigurationForItemAtIndex:(NSInteger)index
+                                                      point:(CGPoint)point
+                                                  viewModel:(id)viewModel API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos);
 
 @end
 

Source/IGListKit/IGListSectionController.h

@@ -139,7 +139,7 @@ NS_SWIFT_NAME(ListSectionController)
  
  @note The default implementation does nothing. **Calling super is not required.**
  */
-- (nullable UIContextMenuConfiguration *)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos);
+- (UIContextMenuConfiguration * _Nullable)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos);
 
 /**
  Identifies whether an object can be moved through interactive reordering.

Source/IGListKit/IGListSectionController.m

@@ -101,7 +101,7 @@ - (void)didHighlightItemAtIndex:(NSInteger)index {}
 
 - (void)didUnhighlightItemAtIndex:(NSInteger)index {}
 
-- (nullable UIContextMenuConfiguration *)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point {
+- (UIContextMenuConfiguration * _Nullable)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point {
   return nil;
 }
 

Source/IGListKit/Internal/IGListAdapter+UICollectionView.m

@@ -284,7 +284,7 @@ - (UIContextMenuConfiguration *)collectionView:(UICollectionView *)collectionVie
   // forward this method to the delegate b/c this implementation will steal the message from the proxy
   id<UICollectionViewDelegate> collectionViewDelegate = self.collectionViewDelegate;
   if ([collectionViewDelegate respondsToSelector:@selector(collectionView:contextMenuConfigurationForItemAtIndexPath:point:)]) {
-      [collectionViewDelegate collectionView:collectionView contextMenuConfigurationForItemAtIndexPath:indexPath point:point];
+      return [collectionViewDelegate collectionView:collectionView contextMenuConfigurationForItemAtIndexPath:indexPath point:point];
   }
 
   IGListSectionController * sectionController = [self sectionControllerForSection:indexPath.section];

Tests/Objects/IGListTestSection.m

@@ -60,7 +60,7 @@ - (void)didUnhighlightItemAtIndex:(NSInteger)index {
     self.wasUnhighlighted = YES;
 }
 
-- (nullable UIContextMenuConfiguration *)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point {
+- (UIContextMenuConfiguration * _Nullable)contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point {
   self.requestedContextMenu = YES;
   return nil;
 }

Tests/Objects/IGTestBindingWithoutDeselectionDelegate.m

@@ -32,7 +32,7 @@ - (void)sectionController:(nonnull IGListBindingSectionController *)sectionContr
 }
 
 
-- (nullable UIContextMenuConfiguration *)sectionController:(nonnull IGListBindingSectionController *)sectionController
+- (UIContextMenuConfiguration * _Nullable)sectionController:(nonnull IGListBindingSectionController *)sectionController
                     contextMenuConfigurationForItemAtIndex:(NSInteger)index
                                                      point:(CGPoint)point
                                                  viewModel:(nonnull id)viewModel API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos) {

Tests/Objects/IGTestDiffingSectionController.m

@@ -65,7 +65,7 @@ - (void)sectionController:(IGListBindingSectionController *)sectionController di
     self.unhighlightedViewModel = viewModel;
 }
 
-- (nullable UIContextMenuConfiguration *)sectionController:(IGListBindingSectionController *)sectionController contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point viewModel:(id)viewModel API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos) {
+- (UIContextMenuConfiguration * _Nullable)sectionController:(IGListBindingSectionController *)sectionController contextMenuConfigurationForItemAtIndex:(NSInteger)index point:(CGPoint)point viewModel:(id)viewModel API_AVAILABLE(ios(13.0)) API_UNAVAILABLE(tvos) {
   self.contextMenuViewModel = viewModel;
   return nil;
 }