[PATCH] : KIO::chmod that uses KURL::List

David Faure dfaure at klaralvdalens-datakonsult.se
Thu Jul 17 13:49:17 BST 2003

On Thursday 17 July 2003 22:40, kzk wrote:
> Hi.
> I implemented KIO::chmod that uses KURL::List in kdelibs/kio/kio/chmod.h, 
> kdelibs/kio/kio/chmod.cpp
> This function is only wrapper.

Your idea sounds good, but I'm afraid the "mask" should be removed, since
there's no way to know the current permissions of the urls - so you can only
use chmod(list of urls) to apply a complete set of permissions to all files,
you can't selectively change only one bit on them (like one can do with the mask).

To be more precise, the current line 79, which uses item->permissions(), will
always get -1 in your case, which will result in wrong stuff being done, unless
m_mask is 0777 (or so).

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

More information about the kde-core-devel mailing list