This fixes an assertion that can occur in an edge case when there is a clashing monster bucket followed by another clashing item. This should improve index/findIndex performance as they no longer need to search these stale links. It's hard to predict if this is significant, but at least it should prevent performance degradation in item repositories over time.
Details
Details
Extended unit test
Diff Detail
Diff Detail
- Repository
- R33 KDevPlatform
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
@olivierjg, I assume from your last comment that you mean to make more changes.
When this is the case, you can select "Changes Planned" (Not sure on the exact language, only visible to owner), as the Action and make a comment. It's useful because then this diff drops out of everyone else's 'action required' list.
Comment Actions
Abandoning for now given inactivity by the original author, for cleaning the to-review list. Still hoping one day someone/you will pick up this again.