<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 13 December 2013 23:34, Daniel Murphy <span dir="ltr"><<a href="mailto:yebblies@nospamgmail.com" target="_blank">yebblies@nospamgmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">"Rikki Cattermole" <<a href="mailto:alphaglosined@gmail.com">alphaglosined@gmail.com</a>> wrote in message<br>

news:otjpativnfoecwjqethp@forum.dlang.org...<br>
<div class="im">><br>
> You have a lot of global variables of same type. With very similar default<br>
> values.<br>
> Example windX and runTmr. They are both doubles.<br>
><br>
> Perhaps an alternative way to write it is like this:<br>
> double<br>
>  windX = 0,<br>
>  runTmr = 0;<br>
><br>
> You also have a lot of enums that like WIDTH and HEIGHT that could be<br>
> transformed into a single enum.<br>
> e.g.<br>
> enum int WIDTH = 800;<br>
> enum int HEIGHT = 600;<br>
><br>
> Would become:<br>
> enum : int {<br>
>  WIDTH = 800,<br>
>  HEIGHT = 600<br>
> }<br>
><br>
> Adds a couple extra lines but hey when you got 20 odd values and repeating<br>
> the type it kinda looks ugly.<br>
><br>
<br>
</div>I would not consider either of those an improvement.<br></blockquote><div><br></div><div>Me either.</div></div></div></div>