Review Request: Remove the Thing class
Vishesh Handa
me at vhanda.in
Fri Jul 27 14:37:12 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105755/
-----------------------------------------------------------
(Updated July 27, 2012, 2:37 p.m.)
Review request for Nepomuk, Release Team and Sebastian Trueg.
Changes
-------
@Release Team (Albert): Please please let me commit this. I hate old stupid code. I'll have to live with it until we release KDE frameworks, that could be years away! If you have to choose any one of my patches to allow - THIS is the one to choose. The bug fixes can always go in 4.9.1. This cannot!
Thanks
Description
-------
No one uses it, and the same class can be generated using the rcgen.
I don't see the point of having this class.
This is my last chance to remove this old code. Otherwise I'll have to wait till KDE Frameworks 5.
Diffs
-----
includes/CMakeLists.txt 4ac2d7c
includes/Thing 952544e
libnepomukcore/CMakeLists.txt 066c898
libnepomukcore/resource/thing.h 2ae3d75
libnepomukcore/resource/thing.cpp 59f2a4d
Diff: http://git.reviewboard.kde.org/r/105755/diff/
Testing
-------
Thanks,
Vishesh Handa
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20120727/633bfd9c/attachment.html>
More information about the release-team
mailing list