[PATCH] D33932: [clang-format] Add support for case-insensitive header matching and use it to improve support for LLVM-style include sorting.

Teodor MICU via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 22 05:28:29 PDT 2017


mteodor added a comment.

In https://reviews.llvm.org/D33932#793994, @djasper wrote:

> Just make clang-format always do this. I don't think anyone is relying on the current behavior.


Well, someone did rely on **case sensitive** match. I thought //case insensitive// by default was just a Windows thing. Not anymore.

  - Regex:           '^"I[A-Z]+[a-z].*\.h"'
    Priority:        1

This rule became too greedy on latest revision. Now not only interfaces are prioritized but also all headers that start with [Ii].


Repository:
  rL LLVM

https://reviews.llvm.org/D33932





More information about the cfe-commits mailing list