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'). -- Marc Mutz <marc.m...@kdab.com> | Senior Software Engineer KDAB (Deutschland) GmbH & Co.KG, a KDAB Group Company Tel: +49-30-521325470 KDAB - The Qt Experts _______________________________________________ Development mailing list Development@qt-project.org http://lists.qt-project.org/mailman/listinfo/development