8a11f0f3a2
The split edges code had a complex method of merging duplicate edges, going backwards to avoid shifting elements in a vector. Sometimes it could result in incorrect corner edge indices though, if it moved an index that matched one of the local variables (I think! I've bee trying to understand this all day and still struggling). Instead, replace it with a `VectorSet` that handles the deduplication by itself, and avoid creating the new edges until the end. I think this code could still be simpler if we tried to reduce the amount of things happening at the same time, making more code deal with the input or final state rather than an in-between one. But to avoid making the change too complicated I stopped here. Pull Request: https://projects.blender.org/blender/blender/pulls/108826
Blender
Blender is the free and open source 3D creation suite. It supports the entirety of the 3D pipeline-modeling, rigging, animation, simulation, rendering, compositing, motion tracking and video editing.
Project Pages
Development
License
Blender as a whole is licensed under the GNU General Public License, Version 3. Individual files may have a different, but compatible license.
See blender.org/about/license for details.