Review Request 116998: Clean up kde4 stuff in knewstuff
Alex Merry
alex.merry at kde.org
Tue Mar 25 14:37:53 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116998/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: knewstuff
Description
-------
Remove reference to KDE4_BUILD_TESTS CMake option
Clean up comments about KConfig
Fix KUrl->QUrl porting
QUrl::setPath() does not behave like KUrl::setPath(). Also removed two
very old FIXMEs: the purpose of the condition is clear now (but it was
probably unnecessary when it used KUrl from kdelibs4), and no-one
noticed any issues with the icon URLs during KDE 4, so KUrl probably did
behave as expected.
Remove completely out-of-date README.kdxs
Diffs
-----
docs/README.kdxs f9843068b964f8e3e0a4fec37f938ceb9bba727e
src/core/engine.cpp 4bcd774e4cdb116fa40e5f2dc7bca2acdeff6224
src/staticxml/staticxmlprovider.cpp 56a25654a70a3628573d85adf4a1c49010f23309
src/uploaddialog.cpp 35f34b30d4dc74c93794cb7862d65364685939d1
tests/README.tests 7b590b823b394ca340e14d6729b4229f49946852
Diff: https://git.reviewboard.kde.org/r/116998/diff/
Testing
-------
Builds, installs.
Thanks,
Alex Merry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140325/475c2af3/attachment.html>
More information about the Kde-frameworks-devel
mailing list