Edited PR text. Added a 3rd topic about future idea for build_heap().
- Python may call this BMesh operator (the UI isn't the only caller).
- The use of a slot instead of
BM_ELEM_SELECTflag check allows the Python API to join non-selected faces.
That…
configure_delimit_data() performs all the functions previously inside bmo_join_triangles_exec in lines 249-302
The logic to join an edge was in the middle of bmo_join_triangles_exec and has also been extracted to a named function for clarity.
the fail: label returned true. removed the gotos and return more clearly.
This logic was originally at the start of bmo_join_triangles_exec() and has been extracted to a standalone function for clarity.
The loop order has been changed here. The vertex order previously returned before was inward-facing. This did not impact the existing code, but is needed for the follow-on work that looks at topology.
This todo was removed. Looking at the logic, I cannot see how to make it any simpler.
6dd42e1b74
Refactor: Change bmo_join_triangles_exec from the use of a sorted list to a priority queue.
