[Konsole-devel] Review Request 128241: Updated version of the file filter patch

Kurt Hindenburg kurt.hindenburg at gmail.com
Wed Jul 6 13:59:47 UTC 2016


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


Ship it!




looks fine - go ahead - not sure why I didn't review the previous patch

- Kurt Hindenburg


On July 4, 2016, 7:53 p.m., Martin Tobias Holmedahl Sandsmark wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128241/
> -----------------------------------------------------------
> 
> (Updated July 4, 2016, 7:53 p.m.)
> 
> 
> Review request for Konsole and Adam Treat.
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> I ported the patch from here: https://git.reviewboard.kde.org/r/114376/
> 
> Original comment:
> Implement a new file filter using POSIX Portable Filename Character Set
> along with KDE's mimetype database of file extension blobs that filters
> text matching a POSIX portable pathname and underlines it very much like
> links.
>     
> Info on the character set allowed can be found at this link:
> http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_267
> 
> 
> Diffs
> -----
> 
>   src/EditProfileDialog.ui 9ff593a 
>   src/Filter.h 8d125f2 
>   src/Filter.cpp 789c4cd 
>   src/SessionController.h 32a9e7c 
>   src/SessionController.cpp b37937d 
> 
> Diff: https://git.reviewboard.kde.org/r/128241/diff/
> 
> 
> Testing
> -------
> 
> Clicked on a lot of files.
> 
> 
> Thanks,
> 
> Martin Tobias Holmedahl Sandsmark
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20160706/680246bf/attachment.html>


More information about the konsole-devel mailing list