Currently the Split Edges & Faces option only split faces whose merged vertex is connected to a single wire edge.
This pach proposes find and use all wire edges linked to the merged vertex are to split faces.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
Event Timeline
| source/blender/editors/mesh/editmesh_select.c | ||
|---|---|---|
| 219–262 | I think co-planar checks like this are too error prone - at different scales we would want different epsilons for subd modeling you may want non-planar faces. Suggest always splitting, if the users connects a wire edge chain and doesn't want to split the face - they can disable this option. | |
| 319 | Should dynamically allocate. | |
| source/blender/editors/mesh/editmesh_select.c | ||
|---|---|---|
| 280 | Stack is normally used for something add and remove (push/pop), this is just an array of edges. Should call r_edgenet r_edgenet_len since these are return args. | |
| 291–296 | Dealing with pointers-to-pointers in the code reads a little awkwardly. This could use local edgenet and edgenet_len variables, then assign the return arguments at the end. | |
Comment Actions
- rename edgenet_stack and stack_len to r_edgenet and r_edgenet_len_max and use local variables