[PATCH] listdir for NetAccess
Simon Hausmann
hausmann at kde.org
Tue Sep 17 08:02:05 BST 2002
On Tue, Sep 17, 2002 at 03:07:52AM +0200, Tobias Koenig wrote:
> Hi,
>
> can I commit the attached patch to CVS HEAD?
> I don't no if it break BC, since I've changed the constructor and
> destructor.
I can't comment on the validity of the patch because I'm a
NetAccess-Hater(tm) :) . Only two minor comments:
1) It looks binary compatible :-)
2) Can you change listdir to listDir, to be consistent with
KIO::listDir?
(the same applies to UDSEntryListToFileNames IMHO, unless it is
supposed to be a private method -- otherwise IMHO we should use
the same default qt/kde naming scheme which starts with a
lower-case word -> udsEntryListToFileNames. As that would look
odd I suggest to add a verb: convertUDSEntryListToFileNames --
possibly it should also be moved to global.h maybe, as it's
unrelated to NetAccess)
Simon
More information about the kde-core-devel
mailing list