pspp-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: commit c70fe01eeffe530040004eae26c655dd1682f1bf


From: John Darrington
Subject: Re: commit c70fe01eeffe530040004eae26c655dd1682f1bf
Date: Tue, 10 Jul 2012 17:05:24 +0000
User-agent: Mutt/1.5.18 (2008-05-17)

On Tue, Jul 10, 2012 at 07:35:10AM -0700, Ben Pfaff wrote:

     > There doesn't seem to be anything initialising de->data_window
     > to NULL so there is still a potential leak here.
     
     A leak, or passing a bad pointer to g_object_unref()?
     
     > However I don't see a need to keep a reference here at
     > all. ->data_window is used only by the do_sort function, and
     > that cannot be called unles the window is realized.
     
     That sounds like a better approach, then, as long as we add a
     comment somewhere that says why we don't need a reference.  Do
     you want to make that change?

Done

-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://keys.gnupg.net or any PGP keyserver for public key.

Attachment: signature.asc
Description: Digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]