[Bf-codereview] Radial control RNA update (issue4280080)
NicholasBishop at gmail.com
NicholasBishop at gmail.com
Sun Apr 3 07:31:45 CEST 2011
Thanks for the detailed review :) I uploaded a new patch to address most
of the issues you raised.
A question about improving the brush texture-getter:
> Another alternative would be to not go through RNA here at all, and
tie these
> textures to ID blocks, and basically pass in the ID block as a
property, and
> have a generic function to get a texture from them. I'd prefer that
personally,
> and it makes it possible to get rid of fairly complex RNA function
calling code
> in the operator.
Do you mean that a GL texture field should be added to the ID struct
itself?
http://codereview.appspot.com/4280080/
More information about the Bf-codereview
mailing list