[PATCH] Make KDiskFreeSpace more convenient -> KJob-like usage

[Sebastian Trüg]

On Wednesday 23 July 2008 22:14:00 Alex Merry wrote:
> On Wednesday 23 July 2008 15:56:29 Sebastian Trüg wrote:
> > On Wednesday 23 July 2008 15:35:52 Dirk Mueller wrote:
> > > On Wednesday 23 July 2008, Sebastian Trüg wrote:
> > > > please find attached a patch for KDiskFreeSpace. I found it to be
> > > > very inconvenient to use.
> > >
> > > A reminescense of the old implementation, which was async. I've changed
> > > it to be synchronous, so I'm not sure if we should invest more in that
> > > old interface. IMHO a static ::getFreeDiskspace() (better method name
> > > welcome) would be enough.
> >
> > Sure, for KDE 5, but for now we have to keep the old API.
>
> That just means you have to keep it around.  You could introduce the static
> method, and mark the old usage as deprecated.

That is what I just wrote to David. So having a method like

bool KIO::freeDiskSpace( const QString& path, KIO::filesize_t* avail, 
KIO::filesize_t* size = 0 )

would that be OK? And if so, should I revert the previous patch?

Cheers,
Sebastian