On Monday, 17 December 2018 22:19:31 CET Heiko Bauke wrote:
> Hi,

Hi Heiko,
 
> currently I am testing some static code analysis tools to search
> systematically for bugs and code smells in the darktable code base.  So
> far, I found a lot of minor issues, which can be easily fixed, and even
> more false positives.

in Fedora we have a nice tool to run different static analysis tools at the 
same time, it is called csbuld.

I'm using it e.g. here:

https://gitlab.com/libssh/libssh-mirror/-/jobs/134349066

It supports checking a git commit range to detect if new issues have been 
introduced. You can tell it to fail if it is the case, that's how I configured 
the runner.

Here is failed because it detected some false positives:

https://gitlab.com/libssh/libssh-mirror/-/jobs/128004825

As a new push checks the new commit range, it sees that they are not newly 
introduced issues and "ignores" them.


We need to modernize cmake in darktable, but I haven't had the time to do that 
yet. It would allow to support some sanitizers by default.


Cheers,


        Andreas


___________________________________________________________________________
darktable developer mailing list
to unsubscribe send a mail to darktable-dev+unsubscr...@lists.darktable.org

Reply via email to