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