| 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: Thomas Huth <th.huth@xxxxxxxxx>
- Date: Sat, 26 Mar 2022 17:45:44 +0000
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.de; s=2017; t=1648316746; bh=0syKZRp3aK7s9omzKyN6OYaBaBsz6QFwZBxMizUX15o=; h=Date:From:To:Subject:From; b=p0DWFTv67RGsaHyxSFBBCqskw9WwnP5+TmGoQ3KNDRm/bs8zQOXQoiUbj9DQqR/3q FiY4XHWVk/F37ccIZVDwuIg10kEBq1kjxFOiH4gkGrOyyrAcSYs+68yI2+4uja40lO lZ7/TVxuQALIOgJoL1Lh7UkyeybPiQOO4vQ5R8bUE8chpDydMbtVQSeFyBQwYJuGfV bl411QNWPMnJ1LDc7McOeoSsV5EYLHr959//jjuNxYSrsDBhTeOXVwY3PoY/nMElcb tFw1iaivr9yQuVWZgI/V1JIwMpMCvSTvtv9XjsT3GtzwJUJnv7ivWnXhgFqZyjHGXf IHFYlOPVTwSJw==
Am Sat, 26 Mar 2022 17:43:27 +0100
schrieb Uwe Seimet <Uwe.Seimet@xxxxxxxxx>:
> Just in case you are interested: These warnings are reported by Eclipse with
> the default settings:
>
> Description Resource Path Location Type
> Suspicious semicolon configuration.c /hatari/src line 854 Code Analysis Problem
> Variables should be declared one per line to improve readability configuration.c /hatari/src line 510 Code Analysis Problem
> Variables should be declared one per line to improve readability dmaSnd.c /hatari/src line 137 Code Analysis Problem
[...]
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.
Thomas