Latest cppcheck changes have been pushed to git [c78da139].
Ted.
On 2/15/25 9:16 AM, Ted Felix wrote:
I've fixed a number of those. Just cast to the right function. Yet
another reason to hate function overloading. There are plenty of
examples in the code. But yeah, the ActionClient one was one I really
wanted to fix but never had time to figure out.
The old SLOT() and SIGNAL() are *really* dangerous. I've been burned
many times. Thanks for working on these. I will be sleeping better at
night. 🤣
Ted.
On 2/14/25 8:05 AM, Philip Leishman via Rosegarden-devel wrote:
So I installed a new system - Opensuse leap 15.6 with a newer cppcheck.
With the old system I had no more errors from cppcheck - now lots !
I will be making some changes to get rid of these warnings.
One thing that cppcheck complains about is with the old style connect -
cppcheck says the slot method is unused. I think we should move to the
new style connect with the method pointer.
One problem here is the FileActionClient where the connect is delegated
to the FileActionClient class. I had to ask Google Gemini how this can
be done - and got a good answer !!!
Changes on my bug-1632-cppcheck branch
Philip
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to
unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
