Suspicious code in revision 867140 (Part 2)

Ingo Klöcker kloecker at kde.org
Sun Oct 5 21:05:56 BST 2008


On Sunday 05 October 2008, Thomas McGuire wrote:
> Hi,
>
> On Saturday 04 October 2008 12:44:10 Christoph Bartoschek wrote:
> > - kdepim/kmail/kmfoldermaildir.cpp:954
> >
> > The comma-operator here?  Maybe the ,0 should be one parenthesis to
> > the left.
>
> Fixed (by removing ,0, I have no idea why that was added at all, the
> function doesn't accept a second parameter)

http://websvn.kde.org/trunk/KDE/kdepim/kmail/kmfoldermaildir.cpp?r1=368633&r2=378470

The ,0 has been left over when KIO::NetAccess::del( foo, 0 ) was 
replaced with removeDirAndContentsRecursively( foo ).


Regards,
Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20081005/3d4ff719/attachment.sig>


More information about the kde-core-devel mailing list