Uh oh!
There was an error while loading. Please reload this page.
made all suppressions in cfg tests inline / added TODOs - #6154
Conversation
I broke this in #5840 by not thinking about the implications of globally suppressing I will look into the TODOs and file tickets later. |
cfg tests inline / added TODOs [skip ci]cfg tests inline / added TODOs| // TODO cppcheck-suppress unreadVariable | ||
| QRegion a; | ||
| // cppcheck-suppress unreadVariable | ||
| // TODO cppcheck-suppress unreadVariable |
There was a problem hiding this comment.
These are most likely https://trac.cppcheck.net/ticket/12366.
| void unreadVariable_QRegion(const int x, const QRegion::RegionType type, const QPolygon &polygon, const QBitmap &bm, const QRegion ®ion, const Qt::FillRule fillRule) | ||
| { | ||
| // cppcheck-suppress unreadVariable | ||
| // TODO cppcheck-suppress unreadVariable |
There was a problem hiding this comment.
I filed https://trac.cppcheck.net/ticket/12529 about these.
| printf(QT_TR_NOOP("Hi")); | ||
| // cppcheck-suppress checkLibraryFunction |
| { | ||
| (void)lgamma(d); | ||
| // cppcheck-suppress invalidFunctionArg | ||
| // TODO cppcheck-suppress invalidFunctionArg |
There was a problem hiding this comment.
This was added in #5845 and was never detected. @orbitcowboy
There was a problem hiding this comment.
Looks like !0.0: doesn't work, e.g. log() has <valid>4.94066e-324:</valid>.
| void unusedAllocatedMemory_std_free() | ||
| { | ||
| //cppcheck-suppress unusedAllocatedMemory | ||
| //TODO cppcheck-suppress unusedAllocatedMemory |
There was a problem hiding this comment.
This was added in #5861 but was never detected. @chrchr-github
firewave
commented
Mar 19, 2024
I have no idea what is going on with |
firewave
commented
Mar 20, 2024
I still do not get what is going on there. I think we should merge this ASAP so we do not regress any further and look into it afterwards. I also want to get #6153 merged first. |
c7af6a2 to
236073cCompare
unmatchedSuppressionwas suppressed globally which obviously has the effect that we got no feedback when an expected warning was not being reported.