search

brettpoole / growl

Automatically exported from code.google.com/p/growl
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Memory leaks in Mist when using the Growl-1.3.1 SDK #517

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Hey guys

While testing my app with the Growl 1.3.1 SDK with no Growl installed it looks 
like Mist notifications are leaking memory. This is under OS X 10.7.4.

Here's what I'm seeing in instruments after generating a few 100 growl 
notifications:

http://f.cl.ly/items/171v0v221Z0F2D0T360s/Mist%20Leaks.png

And the output of the leaks tool after sending one growl notification:

leaks Report Version:  2.0
Process 6813: 17946 nodes malloced for 4302 KB
Process 6813: 5 leaks for 368 total leaked bytes.
Leak: 0x10383a220  size=208  zone: DefaultMallocZone_0x1000f7000   
NSViewAnimation  ObjC  AppKit
    0x73c76640 0x00007fff 0x33333333 0x3fd33333     @f.s....333333.?
    0x3f800000 0x00000000 0x00000000 0x00000000     ...?............
    0x00000000 0x00000000 0xcc5be5b8 0x41b5a053     ..........[.S..A
    0x00000000 0x00000000 0x00000000 0x00000000     ................
    0x00000000 0x00000000 0x00000000 0x00000000     ................
    0x00000100 0x00000000 0x00000000 0x00000000     ................
    0x00000000 0x00000000 0x00000000 0x00000000     ................
    0x00000000 0x00000000 0x0383a200 0x00000001     ................
    ...
Leak: 0x103832510  size=64  zone: DefaultMallocZone_0x1000f7000   
__NSCFDictionary  ObjC  CoreFoundation  item count: 2
Leak: 0x10382f8f0  size=32  zone: DefaultMallocZone_0x1000f7000
    0x00250900 0x00000001 0x00000000 0x00000000     ..%.............
    0x73ccd8e0 0x00007fff 0x00000000 0x00020000     ...s............
Leak: 0x10383a1e0  size=32  zone: DefaultMallocZone_0x1000f7000
    0x73ccd860 0x00007fff 0x00000000 0x00000000     `..s............
    0x73ccd8c0 0x00007fff 0x00000000 0x00040000     ...s............
Leak: 0x10383a200  size=32  zone: DefaultMallocZone_0x1000f7000   __NSArrayI  
ObjC  CoreFoundation  item count: 1

So it's not to major but it would be great to see this fixed in a future 
version of the SDK.

Cheers

John

Original issue reported on code.google.com by j...@aptonic.com on 6 Aug 2012 at 3:55

GoogleCodeExporter commented 8 years ago 
We're currently running a beta of the 2.0 version of the SDK. Would you be 
willing to give that a go? If so, email me chris@growl.info

Original comment by ch...@growl.info on 7 Aug 2012 at 4:03

GoogleCodeExporter commented 8 years ago 
We just released version 2 of the SDK. Drop that in and give it a whirl.

Original comment by ch...@growl.info on 16 Aug 2012 at 12:58

GoogleCodeExporter commented 8 years ago 
I've retested with the Growl 2.0 SDK and can confirm this is fixed. 

I tested on 10.7 (Mist) & 10.8 (Notification Center) and I'm not seeing any 
leaks with Instruments or the leaks tool after generating several 
notifications. Thanks for the fast fix. 

I love the new Notification Center support :) Well done guys, solid release.

Original comment by j...@aptonic.com on 20 Aug 2012 at 3:37

GoogleCodeExporter commented 8 years ago 
Man that's good to hear (especially the NC bit). Closing as fixed. :)

Original comment by ch...@growl.info on 20 Aug 2012 at 1:51