Hold off on the -Weffc++, it is known for false positives. It is based on a set of guidelines for C++ that appear in Scott Meyer's book (first edition!), but these guidelines are routinely violated in well-written code. For example, it warns for any base class having a non-virtual destructor, warns if any member is missing from an initialization list, etc. Some of the advice is inappropriate for modern (C++11) code.