Review Request: Fix compile of KIO with KDE_ENABLE_FINAL enabled.

Nicolas Alvarez nicolas.alvarez at gmail.com
Mon May 2 22:34:45 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101277/
-----------------------------------------------------------

(Updated May 2, 2011, 9:34 p.m.)


Review request for kdelibs.


Changes
-------

Rewrite according to Dawit's suggestion.


Summary (updated)
-------

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.

Fixed by renaming one of the functions.


Diffs (updated)
-----

  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/c922d541/attachment.htm>


More information about the kde-core-devel mailing list