[all-commits] [llvm/llvm-project] 35493b: [clang-format][NFC] Replace deque with vector

Björn Schäpers via All-commits all-commits at lists.llvm.org
Wed Jan 5 03:32:21 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 35493b45603fc897280cfba60866075888d9a790
      https://github.com/llvm/llvm-project/commit/35493b45603fc897280cfba60866075888d9a790
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/UnwrappedLineFormatter.cpp

  Log Message:
  -----------
  [clang-format][NFC] Replace deque with vector

I think the deque was chosen because of a better push_front, but in
combination with llvm::reverse the push_back'ed vector should be the
better choice.

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


  Commit: c2257fe236726ed46f1ec3d68633f1b6663d2513
      https://github.com/llvm/llvm-project/commit/c2257fe236726ed46f1ec3d68633f1b6663d2513
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

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

  Log Message:
  -----------
  [clang-format] Fix SeparateDefinitionBlocks docs and ...

the Style's equality operator.

This amends 6f6f88ffdae1e12e5f950ef418827a77a55c09c7

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


  Commit: a1db43539027898cbd200798c4549115d261ed86
      https://github.com/llvm/llvm-project/commit/a1db43539027898cbd200798c4549115d261ed86
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp

  Log Message:
  -----------
  [clang-format][NFC] Don't pass member by argument

And then use the argument and member.

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


  Commit: 918c977dc1c8905086443e13c172a492247d4709
      https://github.com/llvm/llvm-project/commit/918c977dc1c8905086443e13c172a492247d4709
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp

  Log Message:
  -----------
  [clang-format][NFC] Early return in TokenAnnotator::next

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


  Commit: 29d8535e2b86d27a3c1363646f04ddf1835c30d2
      https://github.com/llvm/llvm-project/commit/29d8535e2b86d27a3c1363646f04ddf1835c30d2
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp

  Log Message:
  -----------
  [clang-format][NFC] TokenAnnotator: Use range based for

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


  Commit: 2ab5d29f556bc3b5510ae495e5bb54b8b5921d1f
      https://github.com/llvm/llvm-project/commit/2ab5d29f556bc3b5510ae495e5bb54b8b5921d1f
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp

  Log Message:
  -----------
  [clang-format][NFC] Use Prev instead of Current->Previous

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


  Commit: 1da96f744951b54f7a3f9bf799fb8e75a31d291b
      https://github.com/llvm/llvm-project/commit/1da96f744951b54f7a3f9bf799fb8e75a31d291b
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp

  Log Message:
  -----------
  [clang-format][NFC] Right.Previous is Left

Use that name. Also remove the one check for its existence, that is
given.

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


  Commit: 8f6af1d4688904fda730d0fea78d2df11252bf40
      https://github.com/llvm/llvm-project/commit/8f6af1d4688904fda730d0fea78d2df11252bf40
  Author: Björn Schäpers <bjoern at hazardy.de>
  Date:   2022-01-05 (Wed, 05 Jan 2022)

  Changed paths:
    M clang/lib/Format/UnwrappedLineFormatter.cpp

  Log Message:
  -----------
  [clang-format][NFC] Put all state change into the for statement

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


Compare: https://github.com/llvm/llvm-project/compare/99c6b12b924f...8f6af1d46889


More information about the All-commits mailing list