[Bf-blender-cvs] [feff4199012] refactor-mesh-position-generic: Remove comment in screw modifier
Hans Goudey
noreply at git.blender.org
Mon Jan 2 19:04:23 CET 2023
Commit: feff4199012e1bada98b33a3b0655840ff280730
Author: Hans Goudey
Date: Mon Jan 2 12:54:51 2023 -0500
Branches: refactor-mesh-position-generic
https://developer.blender.org/rBfeff4199012e1bada98b33a3b0655840ff280730
Remove comment in screw modifier
===================================================================
M source/blender/modifiers/intern/MOD_screw.cc
===================================================================
diff --git a/source/blender/modifiers/intern/MOD_screw.cc b/source/blender/modifiers/intern/MOD_screw.cc
index c0f7b262b1a..a93ebd76b37 100644
--- a/source/blender/modifiers/intern/MOD_screw.cc
+++ b/source/blender/modifiers/intern/MOD_screw.cc
@@ -471,22 +471,6 @@ static Mesh *modifyMesh(ModifierData *md, const ModifierEvalContext *ctx, Mesh *
* Sort edge verts for correct face flipping
* NOT REALLY NEEDED but face flipping is nice. */
- /* Notice!
- *
- * Since we are only ordering the edges here it can avoid mallocing the
- * extra space by abusing the vert array before its filled with new verts.
- * The new array for vert_connect must be at least `sizeof(ScrewVertConnect) * totvert`
- * and the size of our resulting meshes array is `sizeof(MVert) * totvert * 3`
- * so its safe to use the second 2 thirds of #MVert the array for vert_connect,
- * just make sure #ScrewVertConnect struct is no more than twice as big as #MVert,
- * at the moment there is no chance of that being a problem,
- * unless #MVert becomes half its current size.
- *
- * once the edges are ordered, vert_connect is not needed and it can be used for verts
- *
- * This makes the modifier faster with one less allocate.
- */
-
vert_connect = static_cast<ScrewVertConnect *>(
MEM_malloc_arrayN(totvert, sizeof(ScrewVertConnect), __func__));
/* skip the first slice of verts. */
More information about the Bf-blender-cvs
mailing list