[Bf-committers] copy_paint

Campbell Barton ideasman42 at gmail.com
Mon Jun 27 05:26:00 CEST 2011


Since the brush is initialized in readfile with:
p->brush= newlibadr_us(fd, sce->id.lib, p->brush);

I think you're right, though other parts of the code are not adjusting
the user count, asside from RNA which does by defaut.
will look into this further since changes here should really adjust
free_paint and paint_brush_set too.

On Sun, Jun 26, 2011 at 10:59 PM, Jason Wilkins
<jason.a.wilkins at gmail.com> wrote:
> in paint.c we have the function:
>
> void copy_paint(Paint *src, Paint *tar)
> {
>        tar->brush= src->brush;
> }
>
> shouldn't that be this?
>
> void copy_paint(Paint *src, Paint *tar)
> {
>        tar->brush= src->brush;
>        id_us_plus((ID*)(src->brush));
> }
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> http://lists.blender.org/mailman/listinfo/bf-committers
>



-- 
- Campbell


More information about the Bf-committers mailing list