Review Request: Rework MediaSource(const QString &filename) constructor in phonon five.

[Jon Severinsson]

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

(Updated Nov. 1, 2012, 1:48 p.m.)


Review request for KDE Frameworks and Phonon.


Changes
-------

Right you are, new try.


Description
-------

QFSFileEngine has been removed from Qt5, so the MediaSource(const QString &filename) constructor has to be reworked to not need it.  The only behaviourial changes from the original code are that non-local absolute file paths (eg smb share file paths such as "//hostname/share/path") and "qrc:///" URLs actually works.

I have also added qrc support to the QUrl constructor, and deprecated the QString constructor in favour of it.


Diffs (updated)
-----

  phonon/mediasource.h 5374f00 
  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-multimedia/attachments/20121101/fa35f73e/attachment.htm>
-------------- next part --------------
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel at kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel