From kde-commits  Thu May 24 10:10:33 2007
From: David Faure <dfaure () klaralvdalens-datakonsult ! se>
Date: Thu, 24 May 2007 10:10:33 +0000
To: kde-commits
Subject: Re: KDE/kdepimlibs/kabc/plugins/dir
Message-Id: <200705241210.41084.dfaure () klaralvdalens-datakonsult ! se>
X-MARC-Message: https://marc.info/?l=kde-commits&m=118000147509428

Minor comments:

On Thursday 24 May 2007, Laurent Montel wrote:
> +    kdDebug()<<" ir.entryList( QDir::Files ) :"<<dir.entryList( QDir::Files ).count()<<endl;
> +    QStringList lst = dir.entryList( QDir::Files );
"const QStringList lst" to avoid a detach on lst[0] (I think)

> +    if( lst.isEmpty()) //path doesn't exist or list of file empty
> +      return true; 
> +    QString testName = lst[0];
lst.first() 

>     if ( testName.isNull() || testName.isEmpty() ) // no file in directory
>       return true;
can be remove now, your test replaces it 

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