Re: Revisiting coding convention for empty loop body

Hi Adrian,

Although I have no recollection of the discussion you linked to, it looks like using {} is what I suggested back then, and I have not changed my mind since then. In addition of silencing the compiler warning, and being compact, it also in my opinion makes it more obvious to a human reading the code that this is intentional than using a space. So I would also vote for this option.

Thierry

> On 21 Apr 2018, at 13:46, Eugene Sandulenko <[email protected]> wrote:
> 
> On 21 April 2018 at 12:54, Paul Gilbert <[email protected] <mailto:[email protected]>> wrote:
> Hmmm.. if we'd need to do some form of change to existing code to , I'd be more in favor of mandating the use of {} rather than the semicolon, as opposed to requiring an entire comment just to say that it's intentional that the loop doesn't have a body. It would convey the same information, in a more compact form. And it would have the advantage of bypassing the question of whether semicolons should have spaces in front of them.
> 
> I like this idea much more. I would even adjust the conventions to reflect it.
> 
> 
> Eugene 
> _______________________________________________
> Scummvm-devel mailing list
> [email protected]
> http://lists.scummvm.org/listinfo/scummvm-devel

_______________________________________________
Scummvm-devel mailing list
[email protected]
http://lists.scummvm.org/listinfo/scummvm-devel