[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: pixmap leak in KDE 4.5 RC2
From:       Andrei Nistor <coder.tux () ceata ! org>
Date:       2010-07-13 20:50:04
Message-ID: 201007132350.04953.coder.tux () ceata ! org
[Download RAW message or body]

On Tuesday 13 July 2010 23:25:11 Aaron J. Seigo wrote:
> On July 12, 2010, Marco Martin wrote:
> > icon seems to have too many copies that's true)
> 
> the 'i' icon comes from the BusyWidget in the notifications Applet.
> 
> Notifications calls this when a notification is added:
> 
>     Plasma::Animation *pulse =
> Plasma::Animator::create(Plasma::Animator::PulseAnimation, m_busyWidget);
> 
> PulseAnimation creates a WidgetSnapshot. with no parent.
> 
> WidgetSnapshot creates a pixmap snapshot of the widget.
> 
> PulseAnimation never deletes the WdigetSnapshot.
> 
> pixmap leak. lots and lots of 'i's.
> 
> at least that's the theory. i've committed and backported. kisses.

Thanks Aaron. Unfortunately I have to leave to spend part of my holiday with 
my folks and I won't have access to this computer for the next few weeks (I'll 
probably return after 4.5 final is released.)
I'll try to reproduce the bug on my laptop (though I haven't seen any 
weirdness so far) and try a svn build. I'll let you know as soon as I have the 
time.

Thanks again for addressing this so fast, you rock!

Andrei
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]