[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