Creates a Curve with 1 Bezier Spline from 4 positions (start, start handle, end handle, end) and a resolution. The handles are aligned and mirrored automatically.
The default settings recreate the default Bezier Curve
Details
Details
- Reviewers
Hans Goudey (HooglyBoogly) - Maniphest Tasks
- T89220: Curve Primitive Nodes
- Commits
- rB8884d2d61b3d: Geometry Nodes: Curve Primitive Bezier Segment
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Looks good! Just a couple style notes:
- By convention variables should be declared with camel_case
- Values used as floats, like the radius input, etc. should have .0f on the end to show that.
| source/blender/nodes/geometry/nodes/node_geo_curve_primitive_bezier_segment.cc | ||
|---|---|---|
| 23 | I don't blame you for not knowing this, since there's no documentation, but it's PROP_TRANSLATION that gives the node socket units here. Also, above, I don't think PROP_UNSIGNED will do anything, it's just the min and max that matter here. | |
| 67 | /home/hans/Blender-Git/blender/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_bezier_segment.cc: In function ‘std::unique_ptr<CurveEval> blender::nodes::create_bezier_segment_curve(blender::float3, blender::float3, blender::float3, blender::float3, int)’:
/home/hans/Blender-Git/blender/source/blender/nodes/geometry/nodes/node_geo_curve_primitive_bezier_segment.cc:66:19: warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
66 | return std::move(curve);
| ~~~~~~~~~^~~~~~~Basically returning something is optimized by the compiler to just construct the object in the place it's returned to anyway, so std::move is only misleading. | |
| 72 | The limits in the socket template are just soft limits actually, and you can end up with any value if it's plugged into a math node, etc. So a std::max(..., 0); is necessary here. | |
| 78 | Just call this curve I think, that's what I've been calling it everywhere. | |
| source/blender/nodes/geometry/nodes/node_geo_curve_primitive_bezier_segment.cc | ||
|---|---|---|
| 46 | Picky: Suggest start_handle_right to be more consistent with the names in the UI and elsewhere in bezier spline code. | |
| 80 | The minimum resolution for a bezier curve is 1, for 0 there is an assert in set_resolution, so std::max(input, 1); | |
| 83 | Typo (end_ccontrol). Maybe just pass them directly to the function like in the other patch. | |