tar kioslave changes

David Faure dfaure at klaralvdalens-datakonsult.se
Mon Jun 23 11:18:30 BST 2003


On Sunday 22 June 2003 19:23, Laurence Anderson wrote:
> Hi,
> 
> I noticed a bit of duplication between the zip & tar ioslaves, so I've 
> replaced all instances of KTar to KArchive, so the tar ioslave is now generic 
> for any archive type supported by the KArchive classes.
> 
> Please see the attached patch, if its ok I'll add the relevent protocol files 
> & remove the zip ioslave from the makefile.am.

Excellent.
Yay for code factorization and good OO design (KArchive as base class) ;)

Make sure zip:/ fires kio_tar too for compat reasons. 
Should be renamed kio_archive maybe, to avoid confusion?

-- 
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