Il 24/11/19 18:57, Olivier Goffart ha scritto:
The issue is that there is lots of use of these (esp. the most common ones like toString and toInt.) Removing all uses be a huge work for no obvious reasons. (And i was told they are now recommended by clazy)
The reason was avoiding the template bloat of the instantiations of value<T> in client code, while such instantiations exist already in QtCore under a "different form" -- if T is int, QString, etc. then there's toInt(), toString() and so on. (I'm not sure of the overall impact, though.)
If we can use extern templates for the same purpose, then we could drop the clazy warning for Qt 6, keep the toFoo() as porting aids and just make them inline calls to value<Foo>().
My 2 c, -- Giuseppe D'Angelo | giuseppe.dang...@kdab.com | Senior Software Engineer KDAB (France) S.A.S., a KDAB Group company Tel. France +33 (0)4 90 84 08 53, http://www.kdab.com KDAB - The Qt, C++ and OpenGL Experts
smime.p7s
Description: Firma crittografica S/MIME
_______________________________________________ Development mailing list Development@qt-project.org https://lists.qt-project.org/listinfo/development