[PATCH] D47157: Warning for framework headers using double quote includes

Bruno Cardoso Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 23 16:45:04 PDT 2018


bruno added a comment.

Hi Eugene,

> You could just not include new warning into -Wall and -Wextra. Those who will want to check #include statements correctness could enable it explicitly.

This is exactly what's happening in the patch, the new warning isn't part of `-Wall` or `-Wextra`, and is marked `DefaultIgnore`, which means that will fire only when `-Wquoted-include-in-framework-header` is passed to the driver. Am I missing something from your explanation?

Thanks,


Repository:
  rC Clang

https://reviews.llvm.org/D47157





More information about the cfe-commits mailing list