DACA (debian automated code analysis) for mplayer
Hi, is there anything interesting in this report? http://qa.debian.org/daca/cppcheck/sid/mplayer_1.0~rc3++final.dfsg1-1.html Sorry if this topic is already known or inappropriate for the list. Best regards. -- Roberto Ragusa mail at robertoragusa.it
On Fri, Jan 07, 2011 at 11:27:32PM +0100, Roberto Ragusa wrote:
Hi,
is there anything interesting in this report?
http://qa.debian.org/daca/cppcheck/sid/mplayer_1.0~rc3++final.dfsg1-1.html
Sorry if this topic is already known or inappropriate for the list.
Well, it would be better with a recent SVN (rc3 is over a year and half old). Running cppcheck on a recent snapshot shows a few potential issues: http://pastie.org/1439073. It shouldn't be that hard to fix them, so patchs welcome :) -- Clément B.
http://pastie.org/1439073. It shouldn't be that hard to fix them, so patchs welcome :)
I'll do the gui (not win32) ones. Ingo
Running cppcheck on a recent snapshot shows a few potential issues:
These are my gui fixes (ignoring space changes for easy reading, but commit will be with indents). If they are ok, I'll commit them. Ingo
participants (3)
-
Clément Bœsch -
Ingo Brückl -
Roberto Ragusa