Something that came out while looking at #45541: advance_count() in advance_index_iterate() is often called repeatedly as advance_index_iterate() itself is used repeatedly.
Changing that would both be a minor optimization (removal of the funcfion call overhead) and make compiler and other tools better able to analyze the code (now they have to be prepared to the possibility that consequtive advance_count() calls return different results)
Something that came out while looking at #45541: advance_count() in advance_index_iterate() is often called repeatedly as advance_index_iterate() itself is used repeatedly.
Changing that would both be a minor optimization (removal of the funcfion call overhead) and make compiler and other tools better able to analyze the code (now they have to be prepared to the possibility that consequtive advance_count() calls return different results)