[all-commits] [llvm/llvm-project] d5c6b8: Factor out renaming logic from readability-identif...

kepler-5 via All-commits all-commits at lists.llvm.org
Thu Jan 16 13:35:07 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: d5c6b8407c12d39a78f42a216369407cb2d7b511
      https://github.com/llvm/llvm-project/commit/d5c6b8407c12d39a78f42a216369407cb2d7b511
  Author: Logan Smith <logan.r.smith0 at gmail.com>
  Date:   2020-01-16 (Thu, 16 Jan 2020)

  Changed paths:
    M clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp
    M clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.h
    M clang-tools-extra/clang-tidy/utils/CMakeLists.txt
    A clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.cpp
    A clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.h

  Log Message:
  -----------
  Factor out renaming logic from readability-identifier-naming

Before this patch, readability-identifier-naming contained a significant amount
of logic for (a) checking the style of identifiers, followed by (b) renaming/
applying fix-its. This patch factors out (b) into a separate base class so that
it can be reused by other checks that want to do renaming. This also cleans up
readability-identifier-naming significantly, since now it only needs to be
concerned with the interesting details of (a).




More information about the All-commits mailing list