[all-commits] [llvm/llvm-project] 38b1a1: [clang-tidy] allow tests to use --config-file inst...

Alexis Murzeau via All-commits all-commits at lists.llvm.org
Mon Feb 20 11:07:07 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 38b1a17c519d3873f413156c302cb7ed0d39ee3d
      https://github.com/llvm/llvm-project/commit/38b1a17c519d3873f413156c302cb7ed0d39ee3d
  Author: Alexis Murzeau <amubtdx at gmail.com>
  Date:   2023-02-20 (Mon, 20 Feb 2023)

  Changed paths:
    M clang-tools-extra/test/clang-tidy/check_clang_tidy.py
    M clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation-cfgfile.cpp
    M clang-tools-extra/test/clang-tidy/checkers/readability/identifier-naming-hungarian-notation.cpp

  Log Message:
  -----------
  [clang-tidy] allow tests to use --config-file instead of --config

The previous way to test hungarian notation doesn't check CHECK-FIXES.

This will allow readability-identifier-naming tests of Hungarian
notation to keep the use of an external .clang-tidy file (not embedded
within the .cpp test file) and properly check CHECK-FIXES.

Also, it turns out the hungarian notation tests use the wrong
.clang-tidy file, so fix that too to make these tests ok.

This is a part of a fix for issue https://github.com/llvm/llvm-project/issues/60670.

Reviewed By: carlosgalvezp

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




More information about the All-commits mailing list