[PATCH] D74669: [clang-tidy] New check: misc-no-include-cpp

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 17:51:41 PST 2020


Eugene.Zelenko added a comment.

In D74669#1892577 <https://reviews.llvm.org/D74669#1892577>, @jroelofs wrote:

> Implement review feedback:
>
> - Re-purpose `HeaderFileExtensionsUtils.h` to support headers *and* sources.
> - Surround file extension in diagnostic with `''`s.
>
> I have these as two separate patches locally, but I don't know how to represent that in phabricator, so I've uploaded the diff across both. If this gets approved, I intend to push them as separate commits.


You could merge two patches (diff to base version) and update diff in this request.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74669/new/

https://reviews.llvm.org/D74669





More information about the cfe-commits mailing list