create_curves_outline
into grease_pencil_geom.cc
make_new_animlistelem()
by returning early
Yeah the comment isn't quite right. How about
\param use_evaluated_points: Samples are for evaluated points instead of control points.
It means that the sample_indices
refer to a data array…
Looks like
scripts/presets/keyconfig/keymap_data/blender_default.py.orig
made it into this PR by mistake
Oops, fixed.
create_curves_outline
into grease_pencil_geom.cc
Ah yes, correct. My point is: it doesn't affect resolution of the straight sections, which is what i would expect when reading "subdivisions".
create_curves_outline
into grease_pencil_geom.cc
Looks good, just have some suggestions for making parameters more intuitive.
create_curves_outline
into grease_pencil_geom.cc
Comments would be good here (see other comments about changing parameters):
create_curves_outline
into grease_pencil_geom.cc
Rename subdivisions
to cap_subdivisions
, since they only affect cap resolution.
create_curves_outline
into grease_pencil_geom.cc
I'm not sure if the keep_shape
argument makes sense. It just makes it ignore the radius_offset
for the actual outline, in that case the radius is only used for the resulting stroke radius. This is confusing, but i just copied what works for GPv2.
This now uses the shared resample_curve_attribute
method, same as the resampling code.