[PATCH] Add readability-redundant-include check to clang-tidy

Richard legalize at xmission.com
Mon Mar 2 10:43:58 PST 2015


I think there is benefit to checking for redundant includes separately from include order.

On many code bases where I've worked the include order is all over the place.  The main barrier in my experience to having a consistent include order hasn't been one of tools, but getting agreement from the team.  For the llvm/clang code bases the team has already decided on the order of includes, but for many teams that can start a big political battle.

However, noone is going to disagree with eliminating redundant includes.


http://reviews.llvm.org/D7982

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list