Review Request: KIO : Remote preview

Iamluc luc at vieillescazes.net
Wed Jul 14 22:35:49 BST 2010



> On 2010-07-14 21:06:17, Peter Penz wrote:
> > /trunk/KDE/kdelibs/kio/kio/previewjob.cpp, line 353
> > <http://reviewboard.kde.org/r/4627/diff/1/?file=31188#file31188line353>
> >
> >     Only a minor suggestion: I think the code might get slightly more readable when storing the entry.numberValue() first. Also I'd suggest to always initialize scalar types (even we know in this case, that it is initialized after the if for sure).
> >     
> >     bool skipCurrentItem = false;
> >     const KIO::filesize_t size = (KIO::filesize_t)entry.numberValue( KIO::UDSEntry::UDS_SIZE, 0 );
> >     if (d->currentItem.item.url().isLocalFile())
> >     {
> >         skipCurrentItem = !d->ignoreMaximumSize && size > d->maximumLocalSize
> >                           && !d->currentItem.plugin->property("IgnoreMaximumSize").toBool();
> >     }
> >     else
> >     {
> >         // If remote, ignore the "IgnoreMaximumSize" plugin property
> >         skipCurrentItem = !d->ignoreMaximumSize && size > d->maximumRemoteSize;
> >     }

Thanks for your review !

As I don't have a SVN account, I will be glad that you commit this for me. :-)
And please feel free to modify it.

Thanks.


- Iamluc


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4627/#review6566
-----------------------------------------------------------


On 2010-07-14 14:30:11, Iamluc wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4627/
> -----------------------------------------------------------
> 
> (Updated 2010-07-14 14:30:11)
> 
> 
> Review request for kdelibs and Peter Penz.
> 
> 
> Summary
> -------
> 
> Hi,
> 
> There is a size limit for remote previews in Dolphin. By default, the limit is set to 0, so there is no preview for remote files.
> But a ThumbCreator plugin can ignore this limit by setting "IgnoreMaximumSize=true" in the protocol .desktop file (like the ffmpegthumbnails creator).
> 
> This patch forces to ignore this property when using a remote protocol.
> 
> Luc.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/kio/kio/previewjob.cpp 1149793 
> 
> Diff: http://reviewboard.kde.org/r/4627/diff
> 
> 
> Testing
> -------
> 
> Tested with SFTP and FTP kio slaves. Files are not downloaded anymore. 
> 
> 
> Thanks,
> 
> Iamluc
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100714/8c778021/attachment.htm>


More information about the kde-core-devel mailing list