[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