| Re: [hatari-devel] MODE SENSE update patch |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] MODE SENSE update patch
- From: Uwe Seimet <Uwe.Seimet@xxxxxxxxx>
- Date: Sat, 26 Mar 2022 19:04:01 +0100
- Authentication-results: strato.com; dkim=none
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1648317841; s=strato-dkim-0002; d=seimet.de; h=In-Reply-To:References:Message-ID:Subject:To:From:Date:Cc:Date:From: Subject:Sender; bh=zJNw0uox/baYfPfOr7jodaTg5yI1v8v4A/KqgzN0ZOo=; b=RuKuw4YeAKd087vAPHth3ra++c8pND0vgM8qL19Bsr9j4//F8ZCwz+ADMIRC490vW4 Lm3jSLz76eIpvVoXP2GrumhtUeCpwQPE2wP90L6lsTQZ2nzrZX03f79/gOIBhMLMjMOr N9VJ8IkNXSxs/SvQEqtUbEbXWhAh/ZRP2/CnmdsTDcdqQhIy/aF2g+fC8KZuGcPt4RAG BcX01wo3UUVlyAdbVjB+IBrs7UT2dvo7qNEHArUzPngQuEitVzLJM8KA+bfXlem6EQ1F 6HWXtzNcgN8dnxOpD5t9g61co7pYR1KorKzWjUxikJg+qoWA5PhJXxwDddrhPjibz8pL rsFg==
> I agree that the "Suspicious semicolon" could be fixed to help static
> analyzers, but the "one per line" warnings look just like unnecessary
> nitpicking to me, so I suggest that you rather switch them off in your
> project settings.
The warnings do not really bother me, I just posted them in case somebody
was interested. There are also errors, but these are caused by shortcomings
of the CDT code analysis.
It would be interesting to see the analysis results of tools like SonarQube,
which are usually much more intersting. But SonarQube for C is not free,
even though when using sonarcloud.io it may be possible to run an analysis
without needing a SonarQube license. Setting up a SonarQube analysis for C
is rather complicated, though, at least compared to environments where maven
or gradle are used as build tools.
Best regards
Uwe