[all-commits] [llvm/llvm-project] ec8128: [clang-tidy] bugprone-assert-side-effect: Improve ...

[NoQ via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ec812898318036f6c33579b1d35c1a7f83cf112f
      https://github.com/llvm/llvm-project/commit/ec812898318036f6c33579b1d35c1a7f83cf112f
  Author: Artem Dergachev <artem.dergachev at gmail.com>
  Date:   2021-01-28 (Thu, 28 Jan 2021)

  Changed paths:
    M clang-tools-extra/clang-tidy/bugprone/AssertSideEffectCheck.cpp
    M clang-tools-extra/test/clang-tidy/checkers/bugprone-assert-side-effect.cpp

  Log Message:
  -----------
  [clang-tidy] bugprone-assert-side-effect: Improve warning message.

Drop redundant "found", specify what exactly is wrong with side effects
in assert conditions.

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


  Commit: b92a39ac1319c796777bca19a3af2856acbc69c1
      https://github.com/llvm/llvm-project/commit/b92a39ac1319c796777bca19a3af2856acbc69c1
  Author: Artem Dergachev <artem.dergachev at gmail.com>
  Date:   2021-01-28 (Thu, 28 Jan 2021)

  Changed paths:
    M clang-tools-extra/clang-tidy/bugprone/AssertSideEffectCheck.cpp
    A clang-tools-extra/test/clang-tidy/checkers/bugprone-assert-side-effect.m

  Log Message:
  -----------
  [clang-tidy] bugprone-assert-side-effect: Warn on NSAssert by default.

NSAssert and NSCAssert are Objective-C Foundation's standard assert macros.

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


Compare: https://github.com/llvm/llvm-project/compare/6cedffc0ad8d...b92a39ac1319