Review Request 116961: Replace fake MIME type
David Faure
faure at kde.org
Sat Mar 22 11:36:30 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116961/#review53746
-----------------------------------------------------------
Ship it!
Yeah the other hits are just string comparisons for compatibility and don't need the mimetype definition.
- David Faure
On March 21, 2014, 6:07 p.m., Michael Palimaka wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116961/
> -----------------------------------------------------------
>
> (Updated March 21, 2014, 6:07 p.m.)
>
>
> Review request for KDE Frameworks and David Faure.
>
>
> Repository: kio
>
>
> Description
> -------
>
> This is to eventually remove the entry from kde5.xml, which causes shared-mime-info warnings: Unknown media type in type 'all/allfiles'.
>
> There are a few other references to "all/all", "all/allfiles", and "all/" in this repo. I didn't touch them since it wasn't suggested to do so on the mailing list post, and I'm not too familiar with this code.
>
>
> Diffs
> -----
>
> src/filewidgets/kfilefiltercombo.cpp a800b38a50a4cb3c7b89928e09b6be9d8bc19ca0
>
> Diff: https://git.reviewboard.kde.org/r/116961/diff/
>
>
> Testing
> -------
>
> Played with some consumers of filewidgets which seems to behave normally, but again I'm not particularly familiar with what's happening here.
>
>
> Thanks,
>
> Michael Palimaka
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140322/3dc7e37f/attachment.html>
More information about the Kde-frameworks-devel
mailing list