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

List:       kde-commits
Subject:    Re: [kdelibs/frameworks] tier1/kwidgetsaddons/src: Introduce KMessageBoxNotifyInterface
From:       David Faure <faure () kde ! org>
Date:       2013-10-21 16:35:39
Message-ID: 4611700.yt55dIrT5b () asterix
[Download RAW message or body]

On Monday 21 October 2013 15:56:35 Aurélien Gâteau wrote:
>      // TODO use Qt-5.1's Q_GLOBAL_STATIC
>      if (!s_dontAskAgainInterface) {
>          s_dontAskAgainInterface = new KMessageBoxDontAskAgainMemoryStorage;
> }
> +    if (!s_notifyInterface) {
> +        s_notifyInterface = new KMessageBoxNotifyDummy;
> +    }

Could you, while you're there, implement the TODO for both instances?
(to avoid memory leak reports)

Thanks!

-- 
David Faure, faure@kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5

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

Configure | About | News | Add a list | Sponsored by KoreLogic