Suppress ALSA API macros PVS Studio warnings #2783
Conversation
|
Why the MSVC 64-bit build suddenly started exceeding the warning limit? Did the GitHub silently alter the pipeline, or what? |
They did. From VS rev 14.35.xyz to 14.36.abc; looks like a patch update. Suggest bumping the count to match the new detection quantity, and mention the VS rev change (no one's fault, just better warning detection). |
|
OK. I still don't like such changes being made without clear warning and explicit consent from the project. |
|
It was an excellent crusade; almost the half of Peaveestalia has been conquered 📯 🏴 |
Yeah that's a bit dodgy, isn't it... |
|
@kcgen Actually, this was just 1/3 (when I started we had 265 warnings, we are down to 170 now; but few warnings were fixed in the meantime by others), and mostly just the low-hanging fruits. |
The last PR of my PVS Studio crusade - at least for now :)
This time I would like to suppress some warnings, shown on the screenshot below. These are all originating from ALSA headers - the
snd_*are macros, which PVS Studio does not like. I believe there is nothing sane we can do about them (we probably don't want to alter ALSA headers after all), other than ignoring these.