[all-commits] [llvm/llvm-project] b45527: [clang-tidy] Add UseUpperCaseLiteralSuffix option ...

[Da-Viper via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b45527da23ed64a503cc0fd18f63692eb3589748
      https://github.com/llvm/llvm-project/commit/b45527da23ed64a503cc0fd18f63692eb3589748
  Author: Da-Viper <57949090+Da-Viper at users.noreply.github.com>
  Date:   2024-08-27 (Tue, 27 Aug 2024)

  Changed paths:
    M clang-tools-extra/clang-tidy/readability/ImplicitBoolConversionCheck.cpp
    M clang-tools-extra/clang-tidy/readability/ImplicitBoolConversionCheck.h
    M clang-tools-extra/docs/ReleaseNotes.rst
    M clang-tools-extra/docs/clang-tidy/checks/readability/implicit-bool-conversion.rst
    M clang-tools-extra/test/clang-tidy/checkers/readability/implicit-bool-conversion.c
    M clang-tools-extra/test/clang-tidy/checkers/readability/implicit-bool-conversion.cpp

  Log Message:
  -----------
  [clang-tidy] Add UseUpperCaseLiteralSuffix option to readability-implicit-bool-conversion (#104882)

When readability-implicit-bool-conversion-check and
readability-uppercase-literal-suffix-check is enabled this will cause
you to apply a fix twice

from (!i) -> (i == 0u) to (i == 0U) twice instead will skip the middle
one

Adding this option allows this check to be in sync with readability-uppercase-literal-suffix, avoiding duplicate warnings.

Fixes #40544



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications