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

List:       kde-commits
Subject:    Re: KDE/kdelibs/kspell2
From:       Zack Rusin <zack () kde ! org>
Date:       2005-10-31 23:49:12
Message-ID: 200511010049.14158.zack () kde ! org
[Download RAW message or body]

On Thursday 27 October 2005 12:56, David Faure wrote:
> SVN commit 474837 by dfaure:
>
> Improve API by returning a Ptr to avoid leaks or early deletions.
> +        static Broker::Ptr openBroker( KSharedConfig *config = 0 );

Oh, it works? Cool. The problem at least used to be that when you were 
returning Ptr like this it's reference count was going to zero before 
copy. KShared issue, but nevertheless it was making the whole thing 
bomb because the Ptr we were getting after openBroker call had its 
holdee already deleted. I think that one of the initial version of 
those code was returning a Ptr and then I had to revert :)

Zack

-- 
Few women admit their age. Few men act theirs.
[prev in list] [next in list] [prev in thread] [next in thread]