On Fri, Aug 25, 2006 at 07:37:11AM -0300, Damián Viano wrote:
> reopen 369445
> tags = patch
> thanks, control just let's keep my stupid mistake amongst us, ok?
> 
> On Thu, Aug 24, 2006 at 08:33:14PM -0300, Damián Viano wrote:
> >     Perfect, with your instructions I could reproduce and fix this bug. I
> > didn't need to dig so much on who was using the NULL pointer returned
> > by gdk_font_load, since the right fix imho is to avoid overriding the
> > right font in case of not finding the new font appropriate. So this would
> > just use the default font in the gtk theme if the setted font can't be
> > loaded.
> 
> Sory, I made a *stupid* mistake, in that the font gets unref'ed even if
> it's not changed. This would probably throw the same error than before,
> just not so soon, efectivly hidding the bug! Do I rock or what?

I already applied the patch and tested for both cases. Anyway, I see
what you are fixing. I'll apply this one as well and reupload gmoo.
Thanks.

> Anyway, here's a decent patch, on the house ;-P

You do know you that you are sending reverse patches, don't you? ;)

Thanks for solving this,
kind regards,

Paul

-- 
Student @ Eindhoven                         | email: [EMAIL PROTECTED]
University of Technology, The Netherlands   | JID: [EMAIL PROTECTED]
>>> Using the Power of Debian GNU/Linux <<< | GnuPG key ID: 0x50064181

Attachment: signature.asc
Description: Digital signature

Reply via email to