Skip to content

Commit

Permalink
add more details to the error message when catching NSInternalInconsi…
Browse files Browse the repository at this point in the history
…stencyException

Summary: We want to remove this exception handling, but we're seeing quite a lot of `IGFailure` happening recently. Lets improve the error message to make it easier to debug. Once we get down to a reasonable number, we can remove it.

Reviewed By: fabiomassimo

Differential Revision: D52564499

fbshipit-source-id: f70ca2f0f0f8124377eeb2a219878dfa019af87c
  • Loading branch information
Maxime Ollivier authored and facebook-github-bot committed Jan 5, 2024
1 parent d01784b commit 731e894
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Source/IGListKit/Internal/IGListBatchUpdateTransaction.m
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ - (void)_applyDiff:(IGListIndexSetResult *)diffResult {
/// with our product team to properly fix their data source changes outside of the -performBatchUpdatesBlock:
/// IGLisKit processed a new being as an assert that requires investigation,
/// since it will be processed as invalid data source state that needs a reload.
IGFailure(@"The data source returned an invalid number of sections and rows due to updates applied outside of the performBatchUpdates block. This will cause a crash.");
IGFailure(@"IGListKit caught exception (%@): %@", exception.name, exception.reason);
[self begin];
return;
} else {
Expand Down

0 comments on commit 731e894

Please sign in to comment.