Remove dead code in DynArrayIndex.

There’s a dead assignment in DynArrayIndex since this commit.

Also, typeInfo = nil would be an internal error and deserves a crash inside DynArrayIndex, rather than returning nil and crashing the user code anyway.

dai.patch

Alternative version with further refactoring (−15 LoC up from −10, and must be slightly faster thanks to skipping the last elType2 read): dai-v2.patch.

Alternative version with only −14 LoC but crashing on indices = [] instead of returning garbage, and probably even faster for the reason described in #40098: dai-v3.patch.

Edited Mar 10, 2023 by Rika
Assignee Loading
Time tracking Loading