<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Hi Adrian,</div><div class=""><br class=""></div><div class="">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.</div><div class=""><br class=""></div><div class="">Thierry</div><br class=""><div><blockquote type="cite" class=""><div class="">On 21 Apr 2018, at 13:46, Eugene Sandulenko <<a href="mailto:sev.mail@gmail.com" class="">sev.mail@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote">On 21 April 2018 at 12:54, Paul Gilbert <span dir="ltr" class=""><<a href="mailto:paulfgilbert@gmail.com" target="_blank" class="">paulfgilbert@gmail.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="">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.</div></blockquote><div class=""><br class=""></div><div class="">I like this idea much more. I would even adjust the conventions to reflect it.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">Eugene </div></div></div></div>
_______________________________________________<br class="">Scummvm-devel mailing list<br class=""><a href="mailto:Scummvm-devel@lists.scummvm.org" class="">Scummvm-devel@lists.scummvm.org</a><br class="">http://lists.scummvm.org/listinfo/scummvm-devel<br class=""></div></blockquote></div><br class=""></body></html>