It's probably worth restoring the assert for index validity as the alternative is not crashing (which would have been at least somewhat acceptable) but, at least sometimes, silently reading out of bounds memory, as shown by #26148.
Ideal would be to stop using [] (except in loops where index validity is guaranteed — but which should be replaced by range-based for loops and so not use any indices at all) and use at(), but this is probably not going to happen.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()