[all-commits] [llvm/llvm-project] 2adf24: [clang-format] [doc] Fix example of wrapping class...

Emilia Dreamer via All-commits all-commits at lists.llvm.org
Tue Sep 6 03:24:50 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2adf241592b38189994c6c9d2dfbfa391ecbe9fa
      https://github.com/llvm/llvm-project/commit/2adf241592b38189994c6c9d2dfbfa391ecbe9fa
  Author: Passw <passw_passw at outlook.com>
  Date:   2022-09-06 (Tue, 06 Sep 2022)

  Changed paths:
    M clang/docs/ClangFormatStyleOptions.rst
    M clang/include/clang/Format/Format.h

  Log Message:
  -----------
  [clang-format] [doc] Fix example of wrapping class definitions

Example of

BraceWrapping
  AfterClass
is wrong

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


  Commit: 247613548bac55b47eed88e83d5c8640fd4b200a
      https://github.com/llvm/llvm-project/commit/247613548bac55b47eed88e83d5c8640fd4b200a
  Author: Emilia Dreamer <emilia at rymiel.space>
  Date:   2022-09-06 (Tue, 06 Sep 2022)

  Changed paths:
    M clang/lib/Format/UnwrappedLineParser.cpp
    M clang/unittests/Format/FormatTest.cpp
    M clang/unittests/Format/TokenAnnotatorTest.cpp

  Log Message:
  -----------
  [clang-format] Change heuristic for locating lambda template arguments

Previously, the heuristic was simply to look for template argument-
specific keywords, such as typename, class, template and auto
that are preceded by a left angle bracket <.

This changes the heuristic to instead look for a left angle bracket <
preceded by a right square bracket ], since according to the C++
grammar, the template arguments must *directly* follow the introducer.
(This sort of check might just end up being *too* aggressive)

This patch also adds a bunch more token annotator tests for lambdas,
specifically for some of the stranger forms of lambdas now allowed as
of C++20 or soon-to-be-allowed as part of C++23.

Fixes https://github.com/llvm/llvm-project/issues/57093

This does NOT resolve the FIXME regarding explicit template lists, but
perhaps it gets closer

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


Compare: https://github.com/llvm/llvm-project/compare/af72641d2908...247613548bac


More information about the All-commits mailing list