Comment 31 for bug 67129

Revision history for this message
Steve Beattie (sbeattie) wrote :

I've reproduced the bug in the version of notification-daemon included in the Hardy 8.04 release, and have verified that the package in Hardy-proposed definitely improves the memory usage of it -- doing (in bash):

  $ for ((i = 1; i <= 1000; i++)) ; do notify-send -i /usr/share/icons/Human/48x48/places/distributor-logo.png "Message $i" "This is message $i" ; done

caused the -proposed version to use about an additional 7.5M of memory versus 57M for the version released in 8.04 (ia32 versions, the amd64 version in -proposed increases by about 10M). There is still likely more more memory leaks, as the memory usage appears to grow without bound if the for-loop is repeated multiple times, and it would be nice to address those, but the -proposed package is a significant enough improvement to mark this fix as verified.

I don't make extensive use of the notification-daemon, but I noticed no regressions despite the heavy use I was giving it in for loops, plus the occasional application usage.