[Bf-codereview] Small cleanup of unified paint settings (issue 5529077)
NicholasBishop at gmail.com
NicholasBishop at gmail.com
Thu Jan 12 18:23:37 CET 2012
Thanks, committed sans deprecated RNA props.
On 2012/01/12 15:00:57, brechtvl wrote:
> Looks ok, one comment.
http://codereview.appspot.com/5529077/diff/1/source/blender/makesrna/intern/rna_scene.c
> File source/blender/makesrna/intern/rna_scene.c (right):
http://codereview.appspot.com/5529077/diff/1/source/blender/makesrna/intern/rna_scene.c#newcode1656
> source/blender/makesrna/intern/rna_scene.c:1656:
RNA_def_property_ui_text(prop,
> "Sculpt/Paint Use Unified Radius (Deprecated)",
> The manual get/set functions aren't really needed, passing
> "unified_paint_settings.flag" to RNA should work.
> However I don't think deprecated RNA properties need to be kept around
this way,
> especially ones that are unlikely to be used much in scripts, can just
remove
> them.
http://codereview.appspot.com/5529077/
More information about the Bf-codereview
mailing list