[cfe-dev] Any objections to re-sorting #include lines with clang-format/clang-tidy?
Chandler Carruth via cfe-dev
cfe-dev at lists.llvm.org
Fri Mar 11 01:20:24 PST 2016
Yes, anything like this would be a bug in the logic and I'll file it rather
than submit it.
On Fri, Mar 11, 2016 at 10:18 AM Ismail Donmez via cfe-dev <
cfe-dev at lists.llvm.org> wrote:
> On Fri, Mar 11, 2016 at 11:14 AM, Chandler Carruth via cfe-dev
> <cfe-dev at lists.llvm.org> wrote:
> > The crazy clang-format folks have taught it to sort includes, and
> > has a nice way to run it across all of LLVM and Clang.
> > Before I just submit the fixes (and any bug reports to the clang-format
> > folks if it sorts things weirdly) I wanted to double check that folks
> > be ok with this. My understanding is that the sorting should match the
> > conventions, but if not, I'll make sure to get that fixed first.
> Should take care of windows headers, part of DIASupport.h:
> // atlbase.h has to come before windows.h
> #include <atlbase.h>
> #include <windows.h>
> // DIA headers must come after windows headers.
> #include <dia2.h>
> Sorting those would break stuff.
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-dev