Yup. Missed the last two magic numbers in the op definite defaults, thanks for catching them.
A note on performace.
The old logic was O(n+n+n log n) (linear to count all the edges, second linear pass to fill the array, then qsort the array)
New logic is O(n log n) (linear pass to…
5537b668 cada97af At first I thought EDBM_selectmode_flush() was not needed if we had only cleared the selection, but I see it also updates the count of the selected elements. Everything seemed…
cada97af4f
Simplify change logic. Call EDBM_selectmode_flush to ensure selection counts are updated.
b4ee1599 I left it at "Poles by count" and moved the explanation to the tooltip.
https://projects.blender.org/ideasman42/blender/src/branch/pr-128224-update looks good to me.
I think you intend to commit that branch against main, and then I'll merge main into my branch and…
Exactly. I had originally misinterpreted the intentions of the code and introduced a change, now reverted.
This rearranges the original logov to make the flow a little clearer and add some…
Poles Poles and Connected Edges Poles and Connected Faces
?
I dug further into the RNA_property_is_set() behavior and finally figured out what it was doing -- It's determining if the user overrode the default value. 4a8b88fecb restores the previous…