Review Request 118735: Port libkeduvocdocument from KUrl to QUrl
Jeremy Whiting
jpwhiting at kde.org
Wed Jun 18 05:49:12 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118735/
-----------------------------------------------------------
(Updated June 17, 2014, 11:49 p.m.)
Review request for KDE Edu, Aleix Pol Gonzalez and Inge Wallin.
Changes
-------
I think I've addressed everything noted, but I could be wrong. It still builds, but adding some unit tests would be good as a next step.
Repository: libkdeedu
Description
-------
KUrl is deprecated in frameworks, so I've ported libkeduvocdocument away from it, to QUrl.
I also changed the top CMakeLists.txt frameworks version so it will build against frameworks from master branches.
Diffs (updated)
-----
keduvocdocument/keduvocexpression.cpp 343bc73
keduvocdocument/keduvocdocument.cpp 878ccbf
keduvocdocument/keduvocdocument.h a312fc1
keduvocdocument/keduvoccontainer.cpp 23db0fb
keduvocdocument/keduvoccontainer.h 0fc550b
keduvocdocument/autotests/keduvocdocumentvalidatortest.cpp 781bdfa
CMakeLists.txt 21696ef
keduvocdocument/keduvockvtml2reader.h 39545f3
keduvocdocument/keduvockvtml2reader.cpp 0d0c639
keduvocdocument/keduvockvtml2writer.cpp 2a043cf
keduvocdocument/keduvoctranslation.h a836a77
keduvocdocument/keduvoctranslation.cpp 1d7a5eb
keduvocdocument/sharedkvtmlfiles.cpp a5f45a2
keduvocdocument/tests/converter.cpp 126334c
Diff: https://git.reviewboard.kde.org/r/118735/diff/
Testing
-------
It still builds, haven't tried the tests, but I don't think the autotests cover this.
Thanks,
Jeremy Whiting
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140618/97a15f34/attachment-0001.html>
More information about the kde-edu
mailing list