Twitux / Patches / #17 Pixbuf size and memory leak

#17 Pixbuf size and memory leak

Status: closed-accepted

Owner: nobody

Labels: None

Priority: 5

Updated: 2009-02-12

Created: 2009-02-11

Private: No

I revived a patch that fixes a memory leak, and the pixbuf-size-render problem... and needs to be reviewed.

I think the patch it's ok, but needs cleanup.

D.-

Discussion

Daniel Morales - 2009-02-11

combined.avatar.memleaks.svn.diff

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brian Pepple - 2009-02-12

Yeah, the patch does need some clean-up. Daniel, I'll look at it later tonight/tomorrow, if you don't have time.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Daniel Morales - 2009-02-12

I tried the patch tonight; it corrects the avatar size, but has some issues with the renderer.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brian Pepple - 2009-02-12

Cleaned up patch, and commited. svn rev. 372

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brian Pepple - 2009-02-12

status: open --> closed-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Daniel Morales - 2009-02-12

status: closed-accepted --> open-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Daniel Morales - 2009-02-12

status: open-accepted --> closed-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brian Pepple - 2009-02-12

It set the x pad to 50, instead of to 5.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link: