[PATCH] D21323: [clang-format] do not add existing includes.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 14 06:13:52 PDT 2016


ioeric added a comment.

It should be quite straight-forward to de-duplicate #includes in the same block in `sortIncludes`. I'm happy to add this into `sortIncludes`, but I'm not sure if this is a behavior users would expect from clang-format since it deletes code?

And maybe duplicated headers are not that common in the first place? I would rather rely on users (like #include_fixer) to make sure there is no duplicated #include.


http://reviews.llvm.org/D21323





More information about the cfe-commits mailing list