[all-commits] [llvm/llvm-project] 41a09a: [clang-format][NFCish] Alphabetical sort Format.(h...

Björn Schäpers via All-commits all-commits at lists.llvm.org
Tue Nov 8 12:46:51 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 41a09a07ce4ddd1e97ce0430d1debe1dcc853890
      https://github.com/llvm/llvm-project/commit/41a09a07ce4ddd1e97ce0430d1debe1dcc853890
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-11-08 (Tue, 08 Nov 2022)

  Changed paths:
    M clang/docs/ClangFormatStyleOptions.rst
    M clang/include/clang/Format/Format.h
    M clang/lib/Format/Format.cpp
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  -----------
  [clang-format][NFCish] Alphabetical sort Format.(h|pp)

I've:
- Sorted the members of FormatStyle alphabetical. The enums and structs
  are kept close to the member.
- Sorted the yaml io functions, based on the type they operate on.
- Sorted the initializers in getLLVMStyle(), except that penalities are
  kept at the end.
- Sorted the io of FormatStyle, this changes the --dump-config behavior.
- Moved the deprecated options into the only input case, this also
  changes --dump-config, it does not put the not directly used options
  in the .clang-format anymore.
- Sorted the comparisons in operator==.
- Added WhiteSpaceMacros in operator==, I've not actively looked if all
  other members are compared.
- This showed flawed tests (or in my opinion a flawed io operation, but
  that is another discussion and change).

Differential Revision: https://reviews.llvm.org/D137409




More information about the All-commits mailing list