llvm dev - Nov 2015 - Adding some modernize checks to default Clang-tidy configuration

Hi!

I think will be good idea to add some modernize checks to default
Clang-tidy configuration for LLVM, Clang, LLD, Compiler-rt and Clang
extra tools. This will help to maintain code base compliant with
C++11.

Proposed checks are:

modernize-use-override
modernize-use-nullptr
modernize-redundant-void-arg
modernize-shrink-to-fit

I fixed that warnings in aforementioned packages, and only Compiler-rt
still have some because my patches were not merged yet.

Additional efforts are needed to fix other modernize checks warnings.

Eugene.

[Please reply *only* to the list and do not include my email directly
in the To: or Cc: of your reply; otherwise I will not see your reply.
Thanks.]

In article <CAJm83LTzQ95h3W6Lv6HYxoxd=ERA2ZmMyVYeSaXSLQT76gxu0w at
mail.gmail.com>,
    Eugene Zelenko via llvm-dev <llvm-dev at lists.llvm.org> writes:
> I think will be good idea to add some modernize checks to default
> Clang-tidy configuration for LLVM, Clang, LLD, Compiler-rt and Clang
> extra tools. This will help to maintain code base compliant with
> C++11.
> 
> Proposed checks are: [...]
> 
> modernize-redundant-void-arg
I am the author of this check and it is new to clang-tidy.  While I
did my best to cover all the cases I could think of with tests, it is
probably best to be conservative and try it by hand for a while before
adding it to any automatic configuration.

Thoughts?
-- 
"The Direct3D Graphics Pipeline" free book
<http://tinyurl.com/d3d-pipeline>
     The Computer Graphics Museum <http://ComputerGraphicsMuseum.org>
         The Terminals Wiki <http://terminals.classiccmp.org>
  Legalize Adulthood! (my blog) <http://LegalizeAdulthood.wordpress.com>