Review Request 120408: Port textcreator thumbnailer

Christoph Feck christoph at maxiom.de
Sun Sep 28 21:04:58 UTC 2014



> On Sept. 28, 2014, 8:58 p.m., Christoph Feck wrote:
> > thumbnail/CMakeLists.txt, line 110
> > <https://git.reviewboard.kde.org/r/120408/diff/1/?file=315615#file315615line110>
> >
> >     Could you make this line match the previous version (one space after first #) for consistency with the rest?

Ignore that... dropped.


- Christoph


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120408/#review67589
-----------------------------------------------------------


On Sept. 28, 2014, 5:05 p.m., Maarten De Meyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120408/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2014, 5:05 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kio-extras
> 
> 
> Description
> -------
> 
> Quick port. I left the kencodingprober code for now. Maybe we should just drop it? Or fix kencodingprober.
> thumbcreator needs to be a QGuiApplication because we use pixmaps.
> 
> 
> Diffs
> -----
> 
>   thumbnail/CMakeLists.txt aec8b8e 
>   thumbnail/textcreator.cpp d3f633c 
>   thumbnail/thumbnail.cpp 3a38dc9 
> 
> Diff: https://git.reviewboard.kde.org/r/120408/diff/
> 
> 
> Testing
> -------
> 
> Text thumbnails work.
> Utf8 files show with the correct encoding.
> 
> 
> Thanks,
> 
> Maarten De Meyer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140928/23ad2f4c/attachment.html>


More information about the Kde-frameworks-devel mailing list