diff options
| author | Carlos Galvez <carlosgalvezp@gmail.com> | 2023-05-07 17:08:33 +0000 |
|---|---|---|
| committer | Carlos Galvez <carlosgalvezp@gmail.com> | 2023-05-09 16:45:02 +0000 |
| commit | 0d6d8a853a6ea29b5f461a475a8f8eb7e7ba18e2 (patch) | |
| tree | bc170af1400e70a09015166cfda4ea6bf891026d /lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.h | |
| parent | 4d0d295b618edfc937d5bf247f0853df5c70cb96 (diff) | |
[clang-tidy] Fix bugprone-assert-side-effect to actually give warnings
Some time ago a patch was merged to disable all clang-tidy warnings
from system macros. This led to bugprone-assert-side-effect
silently no longer working, since the warnings came from a system
macro. The problem was not detected because the fake assert functions
were implemented in the same file as the test, instead of being a
system include like it's done in the real world.
Move the assert to a proper system header, and fix the code to
warn at the correct location.
This patch is breakdown from https://reviews.llvm.org/D147081
by PiotrZSL.
Fixes https://github.com/llvm/llvm-project/issues/62314
Differential Revision: https://reviews.llvm.org/D150071
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.h')
0 files changed, 0 insertions, 0 deletions