• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Instagram / IGListKit / 5489555151 / 1
99%
master: 100%

Build:
Build:
LAST BUILD BRANCH: update-docs
DEFAULT BRANCH: master
Ran 07 Jul 2023 06:50PM UTC
Files 52
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

07 Jul 2023 06:36PM UTC coverage: 99.831% (-0.07%) from 99.903%
5489555151.1

push

github

facebook-github-bot
fix crash when calling reloadObjects during an update

Summary:
Follow up to D47263265

## What's happening
We're calling `listAdapter.reloadObjects(...)` in the middle of the IGListKit's internal data update, so it's using the new section index path, instead of the old one. So we're going from a list of 2, insert at 0, and try to reload index 1, then we crash (it'll try to update index 2, which doesn't exist in the old array)

## Fix
Just like for `[IGListAdapter performBatchAnimated:updates:completion:]`, lets use the old index. To get this working, lets ask the updater if we're in an update, rather than `IGListAdapter` try to keep track of it.

Sorry the diff is a bit long, but in case this feels reasonable to pick, I might

Differential Revision: D47281472

fbshipit-source-id: 193153cfc

4136 of 4143 relevant lines covered (99.83%)

737.6 hits per line

Source Files on job iOS - 5489555151.1
  • Tree
  • List 0
  • Changed 48
  • Source Changed 48
  • Coverage Changed 25
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 5489555151
  • cd3f84f2 on github
  • Prev Job for on main (#5028149702.1)
  • Next Job for on main (#5607147975.1)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc