Review Request: Fix compile of KIO with KDE_ENABLE_FINAL enabled.
Nicolas Alvarez
nicolas.alvarez at gmail.com
Mon May 2 20:26:18 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101277/
-----------------------------------------------------------
Review request for kdelibs.
Summary
-------
kdelibs compilation is broken with KDE4_ENABLE_FINAL turned ON. The first problem is in kio, where there's two file-static functions with the same name (isLocalRequest) and slightly different code.
This patch fixes kio compilation by moving the two isLocalRequest functions into the corresponding private classes.
Diffs
-----
kio/kio/accessmanager.cpp 03b75898b0fcc6b47518fde8a8b3afc23a1a677f
kio/kio/accessmanagerreply_p.h 408b5331c999633f4f5d603a70cb419678249a7e
kio/kio/accessmanagerreply_p.cpp ccc7474799734b91112d70dc81945b8fe9b9f96c
Diff: http://git.reviewboard.kde.org/r/101277/diff
Testing
-------
Compilation of kdelibs/kio with and without final. Didn't try installing or running.
Other parts of kdelibs still don't work with final.
Thanks,
Nicolas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110502/609eeffd/attachment.htm>
More information about the kde-core-devel
mailing list