Re: [hatari-devel] Remove C++-style comments |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] Remove C++-style comments
- From: Andreas Grabher <andreas_g86@xxxxxxxxxx>
- Date: Sun, 3 Nov 2024 20:21:52 +0100
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1730661718; bh=+QQDCa0Z0ixpI7B2mpfVolyQ+87u/tNx1Ed30uUVrUU=; h=Content-Type:From:Mime-Version:Subject:Date:Message-Id:To: x-icloud-hme; b=Cl7OcRIpiKIHju3FAJW3WwQEIDfEdC9eqMISCssVe7zJebp+Es4Nrl9FdZ3Lr56G1 VzveY7bxOqsrtMyvKNeauYrW+eIhayk6NKTz5yQyC/lpYNgnm87HlCzjPt5wNrUZ6g WLK6oiFbM7JvClMQdcDDvPShEuioI5a2pW36MuhQxgC3wiPdoFJokrKD/aYZTJPrcC MmpNvJIJREjDrmOD/QEyLG5PknNNWtkOI2tt9/jNkLT12LVqhFfIypvl9G6JK8wrRd Vl8wHzfH4AAxqtTmNlEW4f1ABDI7D9nz7YJBMACTB3sNxLBEbhtq3g5KB4a4SpYqwM LqRF8YJzQ3H2g==
> Am 03.11.2024 um 18:41 schrieb Thomas Huth <th.huth@xxxxxxxxx>:
>
> Am Sun, 3 Nov 2024 18:03:15 +0100
> schrieb Vincent Rivière <vincent.riviere@xxxxxxxxxxx>:
>
>>> On 03/11/2024 at 13:17, Thomas Huth wrote:
>>> Honestly, I'm not sure whether we should really enforce the comment markers
>>> in Hatari like this. Using "/* ... */" was important 20 - 30 years ago when
>>> not all C compileres were able to deal with "//" comments yet, but these
>>> times are clearly over.
>>
>> Well, it's just a matter of standard.
>>
>> - If you decide that Hatari is written in C89, then you must forbid //
>> comments, as they aren't part of the standard.
>>
>> - But if you decide that Hatari is written in C99, then you should allow //
>> comments, as they are part of the standard. Except if you want to ban //
>> comments for some other reason. Every project has his own rules.
>
> Hatari uses -std=gnu99. And we don't recommend anything in the coding
> styles, so both ways of comments should be fine. Thus let's really keep the
> comments the way that they currently are.
>
> Thomas
>
Well, I don‘t know what‘s the problem with cleaning this up, especially for single // like in the DSP code (only one // in all files).