On Tue, Jun 6, 2017 at 3:56 AM Chandler Carruth via Phabricator via cfe-commits <cfe-commits@lists.llvm.org> wrote:
> chandlerc created this revision. > Herald added subscribers: mcrosier, sanjoy, klimek. > > This really is a collection of improvements to the rules for LLVM > include sorting: > > - We have gmock headers now, so it adds support for those to one of the > categories. > - LLVM does use 'FooTest.cpp' files to test 'Foo.h' so it adds that suffix > for finding a main header. > - At times the test file's case may not match the header file's case, so > adds support for case-insensitive regex matching of these things. > Curious - where does this last one come up? Can we fix those, or do they look good/intentional/worth keeping? > > With this set of changes, I can't spot any misbehaviors when re-sorting > all of LLVM's unittest '#include' lines. > > Note that I don't know the first thing about testing clang-format, so > please > feel free to tell me what all I need to be doing here. > > > https://reviews.llvm.org/D33932 > > Files: > include/clang/Format/Format.h > lib/Format/Format.cpp > unittests/Format/SortIncludesTest.cpp > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
