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
_______________________________________________
subsurface mailing list
[email protected]
http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface