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

List:       kde-commits
Subject:    Re: branches/kdepim/enterprise/kdepim
From:       David Faure <dfaure () kdab ! net>
Date:       2008-11-28 23:49:26
Message-ID: 200811290049.26851.dfaure () kdab ! net
[Download RAW message or body]

On Wednesday 12 November 2008, Pradeepto Bhattacharya wrote:
>  using namespace KMail;
>  
> +QMap<QString, QString> *KMailICalIfaceImpl::mSubResourceUINamesMap = new QMap<QString, QString>;

This is not portable (and adds noise to memory-leak detectors), you should use K_GLOBAL_STATIC instead
(and then the static is file-static, not defined in the class header)


-- 
David Faure, faure@kde.org, dfaure@kdab.net
KDE/KOffice developer, Qt consultancy projects
Klarälvdalens Datakonsult AB, Platform-independent software solutions

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