Review Request 109516: Deprecate NetAccess::dircopy
David Faure
faure at kde.org
Sat Mar 16 15:16:04 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109516/#review29316
-----------------------------------------------------------
kio/kio/netaccess.h
<http://git.reviewboard.kde.org/r/109516/#comment21909>
Yes, this could become KIO::file_copy + job->ui()->setWindow() + job->exec()
kio/kio/netaccess.h
<http://git.reviewboard.kde.org/r/109516/#comment21908>
... and remove the comment, and add a line to the api docs, saying:
@deprecated since 5.0, use KIO::copy + job->ui()->setWindow() + job->exec() instead
kio/kio/netaccess.h
<http://git.reviewboard.kde.org/r/109516/#comment21910>
And this should be deprecated in favour of KIO::del + the usual.
kio/kio/netaccess.h
<http://git.reviewboard.kde.org/r/109516/#comment21911>
KIO::mkdir + setWindow + exec
staging/kemoticons/src/core/kemoticonsprovider.cpp
<http://git.reviewboard.kde.org/r/109516/#comment21912>
0 is actually the default value, so this isn't useful
- David Faure
On March 16, 2013, 11:29 a.m., Àlex Fiestas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109516/
> -----------------------------------------------------------
>
> (Updated March 16, 2013, 11:29 a.m.)
>
>
> Review request for KDE Frameworks and David Faure.
>
>
> Description
> -------
>
> Patch only adds KDE_NO_DEPRECATED to both NetAccess::dircopy methods and ports KEmoticonsProvider away from it.
>
> I was wondering if I should mark as deprecated some other static methods such file_copy.
>
>
> Diffs
> -----
>
> kio/kio/netaccess.h 3119c74
> kio/kio/netaccess.cpp fd50caa
> staging/kemoticons/src/core/kemoticonsprovider.cpp de073d8
>
> Diff: http://git.reviewboard.kde.org/r/109516/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Àlex Fiestas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130316/a6f6cd0c/attachment.html>
More information about the Kde-frameworks-devel
mailing list