[PATCH] D94201: [clang-format] Skip UTF8 Byte Order Mark while sorting includes

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

In D94201#2483824 <https://reviews.llvm.org/D94201#2483824>, @HazardyKnusperkeks wrote:

> Should the BOM not be ignored in ANY case, not just while sorting includes? And thus a different code should be touched?
>
> How about aligned assignments at the beginning of the line with a BOM, does it count to the type name, or does clang-format handle that correctly right now?

I think its better that issues are kept small and compact and single units of work, lets not make more problems for ourselves by making every bug have to fix every possible permutation.

I follow the "Beyonce Rule" if aligned assignments are an issue when using BOM then the person who added that "should have put a test on it"

But @HazardyKnusperkeks feel free to investigate that if you are interested to see if this issue is further reaching, patches are always welcome


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94201