Expose the List<T>.Enumerator struct or similar about unity-reorderable-list HOT 5 OPEN

One problem with exposing List.Enumerator is that you are going to break ABI compatibility if you switched the underlying collection.

from unity-reorderable-list.

That is true, but is also quite unlikely to happen? And even if the impl. would change it is even more unlikely that anyone will ever declare a variable of this explicit type. i.e. List < XYZ >.Enumerator e = stuff.GetEnumerator(); It is most likely to nearly always be used in foreach loops or if explicitly stored in a variable then by using var e = ...; so for the sake of performance which is paramount in game dev it is imho a risk worth considering to take.

from unity-reorderable-list.

Oh, yeah, no, I completely agree. I was just putting out the possible ABI break so a custom enumerator struct is used that is specific to this type.

That way ABI compat issues can be avoided alltogether by having ReorderableList.Enumerator wrap the underlying list enumerator.

So you can modify implementation details in the future without breaking ABI.

from unity-reorderable-list.

Yes, that would be a sufficient solution, albeit with a small overhead in non optimized scenarios, but still much better than using the IEnumerator. Can we get a comment from the author on this? Should be a 10 min fix.

from unity-reorderable-list.

Hey, I'm super busy with another project right now. If you could add a pull request with the change, I'll approve it

from unity-reorderable-list.