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

List:       kde-core-devel
Subject:    Re: [PATCH] kdelibs: Memory leaks (d-pointer not deleted) fixed
From:       Thiago Macieira <thiago.macieira () kdemail ! net>
Date:       2005-01-04 2:23:10
Message-ID: 200501040023.11428.thiago.macieira () kdemail ! net
[Download RAW message or body]


André Wöbbeking wrote:
>On Friday 31 December 2004 06:07, Benjamin Meyer wrote:
>> Would this be something to be added to the KDELIBS todo list for KDE
>> 4?
>>
>> * Change all FooPrivate *d; -> FooPrivate const *d; and place
>> initialization in constructor.
>
>You probably mean FooPrivate *const d as you want the pointer to be
>const.

Just bear in mind that not all classes would benefit of this. Some delay 
initialisation of d till they need it. Others share d-pointers among 
objects.

-- 
  Thiago Macieira  -  thiago (AT) macieira (DOT) info
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358

4. And æfter se scieppend ingelogode, he wrát "cenn", ac eala! se 
rihtendgesamnung andswarode "cenn: ne wát hú cennan 'eall'. Ástynt."

[Attachment #3 (application/pgp-signature)]

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