Greiner Hormann algorithm.
result.verts.reinitialize(curve.size())
Greiner Hormann algorithm.
It's very helpful to split the code into phases but please, make separation between phases as different functions with meaningful names what is happening on caller.
Greiner Hormann algorithm.
Please do return anything meaningful about the doned one, this just unnecessary hide actual code by meaningless function abstraction. Trying to follow code style from the paper alos does no holds a lot for anyone other.
Greiner Hormann algorithm.
static BooleanResult curve_to_result(const Span<float2> curve)
Greiner Hormann algorithm.
This should be in the header of the whole file imo.
Greiner Hormann algorithm.
Use switch statement for any such case-trees/
Greiner Hormann algorithm.
Do not add const in the header for trivial types.
Greiner Hormann algorithm.
Can we just swap arguments on caller to avoid such case to be in api?
Greiner Hormann algorithm.
Weight is not needed, just call .finalize() after all mixing.
Greiner Hormann algorithm.
So, you can just did 2 loop pass with i and mod(i + 1), but here is one loop with 3 switch and mod(i + 1) for each element?
Greiner Hormann algorithm.
Do the mix here, no reason to wrap this into function.
This is not a bug, object materials is the problem which is need to be removed.
Can't reproduce in/
Broken: version: 4.3.0 Alpha, branch: main, commit date: 2024-08-25 03:57, hash: 6bd515e0d2a2