ThePain/Swiftgram
1
0
Fork 0
mirror of https://github.com/Swiftgram/Telegram-iOS.git synced 2025-12-22 22:25:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rename flag, default it to true, shorten some comment lines

Browse Source
This commit is contained in:
Adlai Holler
2015-09-19 14:21:33 -07:00
parent 7cd79e6e6f
commit 86c1fa3981
1 changed files with 18 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
AsyncDisplayKit/ASCollectionView.mm
View File

@@ -129,12 +129,17 @@ static BOOL _isInterceptedSelector(SEL sel)
BOOL _ignoreMaxSizeChange;
/**
If YES, the `UICollectionView` will reload its data on next layout pass so we should not forward any updates to it.
* If YES, the `UICollectionView` will reload its data on next layout pass so we should not forward any updates to it.
Rationale:
In `reloadData`, a collection view invalidates its data and marks itself as needing reload, and waits until `layoutSubviews` to requery its data source. This can lead to data inconsistency problems. Say you have an empty collection view. You call `reloadData`, then immediately insert an item into your data source and call `insertItemsAtIndexPaths:[0,0]`. You will get an assertion failure saying "Invalid number of items in section 0. The number of items after the update (1) must be equal to the number of items before the update (1) plus or minus the items added and removed (1 added, 0 removed)." The collection view never queried your data source before the update to see that it actually had 0 items.
* Rationale:
* In `reloadData`, a collection view invalidates its data and marks itself as needing reload, and waits until `layoutSubviews` to requery its data source.
* This can lead to data inconsistency problems.
* Say you have an empty collection view. You call `reloadData`, then immediately insert an item into your data source and call `insertItemsAtIndexPaths:[0,0]`.
* You will get an assertion failure saying `Invalid number of items in section 0.
* The number of items after the update (1) must be equal to the number of items before the update (1) plus or minus the items added and removed (1 added, 0 removed).`
* The collection view never queried your data source before the update to see that it actually had 0 items.
*/
BOOL _pendingReloadData;
BOOL _superIsPendingDataLoad;
}
@property (atomic, assign) BOOL asyncDataSourceLocked;
@@ -185,6 +190,8 @@ static BOOL _isInterceptedSelector(SEL sel)
_performingBatchUpdates = NO;
_batchUpdateBlocks = [NSMutableArray array];
_superIsPendingDataLoad = YES;
_collectionViewLayoutImplementsInsetSection = [layout respondsToSelector:@selector(sectionInset)];
_maxSizeForNodesConstrainedSize = self.bounds.size;
@@ -214,7 +221,7 @@ static BOOL _isInterceptedSelector(SEL sel)
{
ASDisplayNodeAssert(self.asyncDelegate, @"ASCollectionView's asyncDelegate property must be set.");
ASDisplayNodePerformBlockOnMainThread(^{
_pendingReloadData = YES;
_superIsPendingDataLoad = YES;
[super reloadData];
});
[_dataController reloadDataWithAnimationOptions:kASCollectionViewAnimationNone completion:completion];
@@ -412,7 +419,7 @@ static BOOL _isInterceptedSelector(SEL sel)
- (NSInteger)numberOfSectionsInCollectionView:(UICollectionView *)collectionView
{
_pendingReloadData = NO;
_superIsPendingDataLoad = NO;
return [_dataController numberOfSections];
}
@@ -648,7 +655,7 @@ static BOOL _isInterceptedSelector(SEL sel)
- (void)rangeController:(ASRangeController *)rangeController endUpdatesAnimated:(BOOL)animated completion:(void (^)(BOOL))completion {
ASDisplayNodeAssertMainThread();
if (!self.asyncDataSource || _pendingReloadData) {
if (!self.asyncDataSource || _superIsPendingDataLoad) {
if (completion) {
completion(NO);
}
@@ -700,7 +707,7 @@ static BOOL _isInterceptedSelector(SEL sel)
{
ASDisplayNodeAssertMainThread();
if (!self.asyncDataSource || _pendingReloadData) {
if (!self.asyncDataSource || _superIsPendingDataLoad) {
return; // if the asyncDataSource has become invalid while we are processing, ignore this request to avoid crashes
}
@@ -719,7 +726,7 @@ static BOOL _isInterceptedSelector(SEL sel)
{
ASDisplayNodeAssertMainThread();
if (!self.asyncDataSource || _pendingReloadData) {
if (!self.asyncDataSource || _superIsPendingDataLoad) {
return; // if the asyncDataSource has become invalid while we are processing, ignore this request to avoid crashes
}
@@ -738,7 +745,7 @@ static BOOL _isInterceptedSelector(SEL sel)
{
ASDisplayNodeAssertMainThread();
if (!self.asyncDataSource || _pendingReloadData) {
if (!self.asyncDataSource || _superIsPendingDataLoad) {
return; // if the asyncDataSource has become invalid while we are processing, ignore this request to avoid crashes
}
@@ -757,7 +764,7 @@ static BOOL _isInterceptedSelector(SEL sel)
{
ASDisplayNodeAssertMainThread();
if (!self.asyncDataSource || _pendingReloadData) {
if (!self.asyncDataSource || _superIsPendingDataLoad) {
return; // if the asyncDataSource has become invalid while we are processing, ignore this request to avoid crashes
}