[all-commits] [llvm/llvm-project] ece7e9: [clang-format] NFC - clang-format the FormatTests

MyDeveloperDay via All-commits all-commits at lists.llvm.org
Sat May 2 07:42:50 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ece7e95f02cdcdf9fa7c6fdfbf6db92e4cc0c87e
      https://github.com/llvm/llvm-project/commit/ece7e95f02cdcdf9fa7c6fdfbf6db92e4cc0c87e
  Author: mydeveloperday <mydeveloperday at gmail.com>
  Date:   2020-05-02 (Sat, 02 May 2020)

  Changed paths:
    M clang/unittests/Format/CleanupTest.cpp
    M clang/unittests/Format/FormatTestCSharp.cpp
    M clang/unittests/Format/FormatTestComments.cpp
    M clang/unittests/Format/FormatTestJS.cpp
    M clang/unittests/Format/FormatTestJava.cpp
    M clang/unittests/Format/FormatTestObjC.cpp
    M clang/unittests/Format/FormatTestProto.cpp
    M clang/unittests/Format/FormatTestRawStrings.cpp
    M clang/unittests/Format/FormatTestSelective.cpp
    M clang/unittests/Format/FormatTestTextProto.cpp
    M clang/unittests/Format/NamespaceEndCommentsFixerTest.cpp
    M clang/unittests/Format/SortIncludesTest.cpp
    M clang/unittests/Format/UsingDeclarationsSorterTest.cpp

  Log Message:
  -----------
  [clang-format] NFC - clang-format the FormatTests

Summary:
Ensure the clang-format unit tests are themselves clang-formatted

Having areas of the llvm code which are clang-format clean, give us more areas to run new clang-format binaries on ensuring we haven't broken anything.

It seems to me we SHOULD have this clang-formatted at a minimum, otherwise how can we expect others to use clang-format if we "don't eat our own dogfood", also if the tests are dependent on the formatting of the code then that would also be bad!

Reviewed By: sammccall

Subscribers: cfe-commits

Tags: #clang, #clang-format

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




More information about the All-commits mailing list