Review Request 122621: Implemented SlaveBase::GetFileSystemFreeSpace in sftp ioslave
Emmanuel Pescosta
emmanuelpescosta099 at gmail.com
Wed Feb 25 10:31:33 UTC 2015
> On Feb. 25, 2015, 11:24 a.m., Vishesh Handa wrote:
> > sftp/kio_sftp.cpp, line 247
> > <https://git.reviewboard.kde.org/r/122621/diff/1/?file=350141#file350141line247>
> >
> > The code indentation seems wrong.
Just followed the coding style (1 tab -> 2 spaces) from kio_sftp.cpp
Should I use the frameworks coding style instead?
- Emmanuel
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122621/#review76592
-----------------------------------------------------------
On Feb. 18, 2015, 12:14 p.m., Emmanuel Pescosta wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122621/
> -----------------------------------------------------------
>
> (Updated Feb. 18, 2015, 12:14 p.m.)
>
>
> Review request for KDE Frameworks and Andreas Schneider.
>
>
> Repository: kio-extras
>
>
> Description
> -------
>
> Implemented SlaveBase::GetFileSystemFreeSpace to enable free space information in sftp ioslave.
>
>
> Diffs
> -----
>
> sftp/kio_sftp.h e088472
> sftp/kio_sftp.cpp 057fcd4
>
> Diff: https://git.reviewboard.kde.org/r/122621/diff/
>
>
> Testing
> -------
>
> Works
>
>
> Thanks,
>
> Emmanuel Pescosta
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150225/62a7d720/attachment.html>
More information about the Kde-frameworks-devel
mailing list