source/blender/editors/gpencil/gpencil_paint.c
1207	Maybe is better move both elements of the if/else to two static functions to keep more clear. if Bezier do bezier_func else do_other_func
source/blender/makesrna/intern/rna_brush.c
1858	Always use Uppercase: `Bezier Smoothing`. Not sure we need "Use"

@Matias Mendiola (mendio) @Daniel Martinez Lara (pepeland) Could you test the patch?

@Matias Mendiola (mendio) Maybe it is better use an Enum with the smooth types instead to put a checkbox. Something like "Stroke and Bezier"

Hans Goudey (HooglyBoogly) added a subscriber: Hans Goudey (HooglyBoogly).Nov 14 2020, 8:07 PM

Hans Goudey (HooglyBoogly) added inline comments.

source/blender/makesrna/intern/rna_brush.c
1858	Currently the `.` is added automatically to the end of the description, so don't bother with that. Also, here is a different wording that I'd suggest (just a bit more actively phrased): `"Uses bezier curve fitting as a smoothing step."` -> `"Fit the stroke to a bezier curve as a smoothing step"`

I would like to drop the "Bezier" in the UI altogether (really, it should be "Bézier"). I think for the user "Curve Smoothing" would be better. What do you think? Also I like the idea of choosing the option from a drop down (though I would not be sure on how to call the default smoothing? maybe just "Default Smoothing"?). This way we can also expose different options for the two methods (the curve smoothing needs some other options that are hardcoded now).

Max Cajazeiras (blackviking) awarded a token.Nov 19 2020, 12:46 PM

I have tried it and it works very well, only two things to improve it IMO:

Using a dropdown is right, but I wouldn't use "Default" because it is not descriptive, rather I would use something like "Smooth type: Simple / Bezier"
The smooth value needed to get a similar final line result differ a lot between both methods. The bezier method needs very small values (0.05 vs 0.50). I would try to get similar final line precision with similar values in both methods.