Review Request: Remove QFSFileEngine useage from Phonon five.

[Jon Severinsson]

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

(Updated Oct. 21, 2012, 9:59 a.m.)


Review request for KDE Frameworks and Phonon.


Changes
-------

Harald, what about this patch?

The new constructor should work correctly with Qt resources using file path (:/)
or URL (qrc://) syntax, valid URLs (including file:/// and file://hostname/ URLs),
relative file paths (relative to cwd), absolute file paths ("D:/", "D:\\", "/"),
and smb share file paths ("//hostname/", "\\\\hostname\\"). I dont't think
file://hostname/ URLs (and thus smb share file paths) can be played on platforms
other than windows (I know KIO don't support them), but that is not really a
phonon issue.

Unlike the old constructor, paths to non-existent files will work just like
URLs to non-existent files, e.g. it will result in a valid MediaSource with
a valid URL, which will fail once you try to actually play it.

Also unlike the old constructor, file::/// URLs will be considered a
MediaSource::LocalFile source, while a smb file path will be considered
a MediaSource::Url source. I did this as I think MediaSource::type()
should reflect whether the file returned by MediaSource::fileName() can
be opened rather than what syntax was used to create the MediaSource.


Description
-------

QFSFileEngine has been removed from Qt5, so rip it out of phonon/mediasource.cpp. Any pointers to a propper fix would be welcome.


Diffs (updated)
-----

  phonon/mediasource.cpp 9e35094 

Diff: http://git.reviewboard.kde.org/r/106974/diff/


Testing
-------


Thanks,

Jon Severinsson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20121021/36c0fc79/attachment.html>