On 24/07/15 13:34, "Marc Mutz" <marc.m...@kdab.com> wrote: >On Thursday 23 July 2015 16:29:12 Knoll Lars wrote: >> On 23/07/15 14:27, "Marc Mutz" <marc.m...@kdab.com> wrote: >> >On Thursday 08 January 2015 23:33:34 Thiago Macieira wrote: >> >> * Q_DECL_OVERRIDE - required in new code, don't add to old code >> > >> >- Required to be used consistently within a class, because Clang warns >>be >> > >> > default if one function has override and another doesn't. >> > >> >- Even add to old code. The benefits outweigh the git history churn. >> > >> >Consensus? >> >> Yes, but again, I’d propose we wait until 5.6 is branched and then start >> using the keyword directly, not the macro. > >Is there any compiler we support that doesn't support it? Note: MSVC >supports >it from 2005 (override; final was called 'sealed').
I believe they all support it, but in 5.6, we still allow compilation in C++98 mode, so it has to be a macro there. Cheers, Lars _______________________________________________ Development mailing list Development@qt-project.org http://lists.qt-project.org/mailman/listinfo/development