[all-commits] [llvm/llvm-project] e73094: [clang-format] Suppress diagnostics on second parse

Tim Wojtulewicz via All-commits all-commits at lists.llvm.org
Fri Mar 5 12:43:31 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e7309404536cf7f389afc8678688f9e3726f1eb8
      https://github.com/llvm/llvm-project/commit/e7309404536cf7f389afc8678688f9e3726f1eb8
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2021-03-05 (Fri, 05 Mar 2021)

  Changed paths:
    M clang/include/clang/Format/Format.h
    M clang/lib/Format/Format.cpp

  Log Message:
  -----------
  [clang-format] Suppress diagnostics on second parse

This amends 25f753c51e7b17bfca08155c1d777c5667110970.

When applying the child configurations we don't need any diagnostic,
because it was issued when first parsing them. So just drop everything
on the second parse.

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


  Commit: 7b02794f0aeb7eaf68d18fffc3057c3c7d975d75
      https://github.com/llvm/llvm-project/commit/7b02794f0aeb7eaf68d18fffc3057c3c7d975d75
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2021-03-05 (Fri, 05 Mar 2021)

  Changed paths:
    M clang/docs/ClangFormatStyleOptions.rst
    M clang/docs/ReleaseNotes.rst
    M clang/include/clang/Format/Format.h
    M clang/lib/Format/Format.cpp
    M clang/tools/clang-format/ClangFormat.cpp
    M clang/unittests/Format/FormatTest.cpp
    M clang/unittests/Format/SortIncludesTest.cpp

  Log Message:
  -----------
  [clang-format] Rename case sorting

As discussed in D95017 the names case sensitive and insensitive should
be switched.

This amends a8105b3766e4.

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


  Commit: f7f9f94b2e2b4c714bac9036f6b73a3df42daaff
      https://github.com/llvm/llvm-project/commit/f7f9f94b2e2b4c714bac9036f6b73a3df42daaff
  Author: Tim Wojtulewicz <timwoj at gmail.com>
  Date:   2021-03-05 (Fri, 05 Mar 2021)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/lib/Format/UnwrappedLineFormatter.cpp
    M clang/lib/Format/UnwrappedLineParser.cpp
    M clang/lib/Format/UnwrappedLineParser.h
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  -----------
  [clang-format] Rework Whitesmiths mode to use line-level values in UnwrappedLineParser

This commit removes the old way of handling Whitesmiths mode in favor of just setting the
levels during parsing and letting the formatter handle it from there. It requires a bit of
special-casing during the parsing, but ends up a bit cleaner than before. It also removes
some of switch/case unit tests that don't really make much sense when dealing with
Whitesmiths.

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


Compare: https://github.com/llvm/llvm-project/compare/c2de5aff1a08...f7f9f94b2e2b


More information about the All-commits mailing list