summaryrefslogtreecommitdiff
path: root/README.Code
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2015-02-07 10:15:30 +0100
committerStephan Bergmann <sbergman@redhat.com>2015-02-07 12:35:55 +0100
commitd2e9b222d7025c53aa82e5718418ea712d21ac3c (patch)
tree163c39a9c94af0d9460e6d3f576a5021954f8ee5 /README.Code
parent905d4db48a0b98f540c8abc3e12fb80be4826029 (diff)
loplugin:deletedspecial to help add SAL_DELETED_FUNCTION annotations
...to special member function declarations that were left undefined. Helps compilers do a better job at identifiying unused class members. This plugin uses heuristics and whitelists do identify applicable declarations, but is not appropriate for "unattended" use so is placed into store/. The following commits contain the results of running this plugin, per module: * Declarations of undefined special member functions are mmarked SAL_DELETED_FUNCTION (aka "= delete", which is deemed superior to deriving the class from boost::noncopyable, cf. Howard Hinnant's reply to <http://stackoverflow.com/questions/7823990/what-are-the-advantages-of-boostnoncopyable>. * Any redundant "explicit" or SAL_DLLPRIVATE markers are removed from the deleted definitions. * Some redundant declarations of undefined default ctors are simply removed; smelled like clueless cargo-cult to have them declared at all. * Some declarations of undefined operator == etc. are left in (and marked SAL_DELETED_FUNCTION) for now, to be on the safe side, though they are likely clueless cargo-cult, too. * Most "static-only" classes are replaced with namespaces (and some where that would be non-trivial due to private members are marked TODO for later). * Newly identified unused class members are removed. Change-Id: Ibeaae4fd579d7a0971a2c2a654a2263acd13414a
Diffstat (limited to 'README.Code')
0 files changed, 0 insertions, 0 deletions