[all-commits] [llvm/llvm-project] 45ddc1: [clang-tidy][NFC] Remove ModernizeTidyModule::getM...

Carlos Galvez via All-commits all-commits at lists.llvm.org
Wed Feb 15 02:38:53 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 45ddc157ca7c464ebbea627da0f682b1554e2390
      https://github.com/llvm/llvm-project/commit/45ddc157ca7c464ebbea627da0f682b1554e2390
  Author: Carlos Galvez <carlosgalvezp at gmail.com>
  Date:   2023-02-15 (Wed, 15 Feb 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/ModernizeTidyModule.cpp
    M clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp

  Log Message:
  -----------
  [clang-tidy][NFC] Remove ModernizeTidyModule::getModuleOptions

Most of the options stated there are duplicated already in
the implementation of each check as a default value for
each option.

The only place where this is not the case is the nullptr
check. Move the default option there instead. Only the
HICPP guidelines alias this modernize check, and there is
nothing in the documentation that suggests it should have
a different default value than the main modernize check.

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




More information about the All-commits mailing list