Review Request 116012: Inline deprecated methods, and silence false positives in tests
Alex Merry
alex.merry at kde.org
Fri Feb 28 00:58:19 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116012/
-----------------------------------------------------------
(Updated Feb. 28, 2014, 12:58 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks.
Repository: karchive
Description
-------
Four commits (in git log order):
Simplify autotests CMakeLists.txt
Separate out tests of deprecated functions to their own file
This allows us to disable deprecation warnings just for that file.
Inline deprecated methods
This both documents how to replace them in the headers and ensures that
enabling or disabling deprecated functions does not affect the ABI.
Use KARCHIVE_NO_DEPRECATED instead of KDE_NO_DEPRECATED
Diffs
-----
autotests/CMakeLists.txt 1da853fe49ec6fca40ccf417db788340124b41e5
autotests/deprecatedtest.cpp PRE-CREATION
autotests/karchivetest.cpp 6d6b5848ce9add84736968dd13f529711f895493
src/karchive.h d4209cf334190dda735fcb4687fa102a4e7a73cd
src/karchive.cpp a25b85e41618d1f7ae62e7e41966afe1a8aaef8e
src/kfilterdev.h 4a35879a15992d19378c7e9baa9b206e187713fd
src/kfilterdev.cpp 60660a0c08922ee0f4ddaf4ad4d1adf0e7f1c5a3
Diff: https://git.reviewboard.kde.org/r/116012/diff/
Testing
-------
Builds, tests pass. No deprecation warnings (before there were two in the tests).
Thanks,
Alex Merry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140228/cee4ea9a/attachment.html>
More information about the Kde-frameworks-devel
mailing list