serenity icon indicating copy to clipboard operation
serenity copied to clipboard

CI+CMake: Enable Clang plugins for Serenity Clang builds

Open ADKaster opened this issue 1 year ago • 1 comments

:partying_face:

As a note, the Lambda capture one is very noisy.

And the warnings from these plugins don't seem to respect -Werror which is... good(?)

ADKaster avatar May 14 '24 00:05 ADKaster

As a note, the Lambda capture one is very noisy.

Yeah I'm going to invert this so that functions that do escape are annotated. After that, I can make the plugins emit errors instead of just warnings. Not sure how useful running this on CI will be until that happens, but I suppose it can't hurt (assuming there are no performance impacts, which I haven't tested!)

mattco98 avatar May 14 '24 01:05 mattco98

I'll drop the CI commit until we can have the warnings enforced :)

ADKaster avatar May 14 '24 18:05 ADKaster