static instance() method removal, header guards patches
Thiago Macieira
thiago at macieira.org
Tue Feb 11 14:15:51 UTC 2014
Em ter 11 fev 2014, às 19:26:20, Boris Barbulovski escreveu:
> Here are three patches:
>
> 1. Removal of static QStringListModel::instance() method, and utilize
> MainTab private variable members.
> Reasons of this are:
> * Simplifying the code.
> * Performance, less dynamic objects.
> * Allows second instance of modified classes for future needs.
Yes, please! Looks good, thanks for doing it.
> 2. Small patch that adds/removes '#include <QStringListModel>' statement.
>
> 3. Put include guards and guard end comment to every header.
>
> br, Boris..
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
Software Architect - Intel Open Source Technology Center
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
More information about the subsurface
mailing list