*picky* prefer brackets when mixing add/subtract with multiply/divide. Bugs caused by edits that don't take order of operations into account happen from time to time, so better avoid them entirely.
This function should also create a Heap for BLI_polyfill_beautify in addPolygon to use to avoid re-creating the Heap for every faces with >3 vertices.
Shouldn't CONCAVE_HOLE be the default? (as far as I was aware this is what users want practically all the time).
Re-opening as the fix will only be enabled once Linux libs uses Wayland v1.22.
Generally seems fine - minor issues noted inline.
Testing I ran into two issues though:
Mesh::mlooppointer is left set after writing a file, shouldn't this be…
Generally seems fine - minor issues noted inline.
*picky* renames input_ to src_, a prefix which remains used elsewhere. Use input_ here too?
*picky*: odd not to call this vert_i ? (used elsewhere).