Review Request #121080 - Created Nov. 8, 2014 and updated
|Andrius da Costa Ribas|
|kde-baseapps, kdeframeworks, kdewin|
Since we're not using kdemacros.h here anymore, KDE_DUMMY_QHASH_FUNCTION is not available. Implement it instead.
It builds (MSVC2013 - 64bit) after this patch (along other patches I'm sending to review today). Kdebase-apps is still not very functional, though (missing icons and weird UI).
Just wondering: the place I found in Qt which does something similar (dummy qHash for MSVC), doesn't have any ifdef: qitemselectionmodel.h. Any idea why we have one here?
It seems to me that this is useful for both the lib and the app, since they both see the QList<KonqHistoryEntry> (whose toSet method requires a qHash implementation)
const ref, no?