<div class="gmail_quote">On Wed, Jul 1, 2009 at 9:00 AM,  <span dir="ltr"><<a href="mailto:dkasak13@users.sourceforge.net">dkasak13@users.sourceforge.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Revision: 41996<br>
          <a href="http://scummvm.svn.sourceforge.net/scummvm/?rev=41996&view=rev" target="_blank">http://scummvm.svn.sourceforge.net/scummvm/?rev=41996&view=rev</a><br>
Author:   dkasak13<br>
Date:     2009-07-01 16:00:27 +0000 (Wed, 01 Jul 2009)<br>
<br>
Log Message:<br>
-----------<br>
Made coordinate specification mandatory when constructing objects of type Sprite and Class. Made transforming from columnwise a default (since it was done most of the time anyway). Changed coordinates to use uint instead of uint16.<br>


<br>Modified: scummvm/branches/gsoc2009-draci/engines/draci/sprite.h<br>
===================================================================<br>
--- scummvm/branches/gsoc2009-draci/engines/draci/sprite.h      2009-07-01 15:22:36 UTC (rev 41995)<br>
+++ scummvm/branches/gsoc2009-draci/engines/draci/sprite.h      2009-07-01 16:00:27 UTC (rev 41996)<br>
@@ -43,19 +43,19 @@<br>
        virtual uint16 getWidth() { return _width; }<br>
        virtual uint16 getHeight() { return _height; }</blockquote><div><br></div><div>you still have it above</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

 public:<br>
-       Sprite(byte *raw_data, uint16 width, uint16 height, uint16 x = 0, uint16 y = 0,<br>
-               bool columnwise = false);<br>
+       Sprite(byte *raw_data, uint16 width, uint16 height, uint x, uint y,<br>
+               uint z, bool columnwise = true);<br>
+<br>
<br>
-       Sprite(byte *sprite_data, uint16 length, uint16 x = 0, uint16 y = 0,<br>
-               bool columnwise = false);<br>
+       Sprite(byte *sprite_data, uint16 length, uint x, uint y,<br>
+               uint z, bool columnwise = true);</blockquote><div><br></div><div>and here</div><div> </div><div>I suggest you to use grep to find the other occurrences.  while it may still make sense to keep uint16 in structs to save memory, these types probably better shouldn't occur as parameters or return values, because the CPU's simply don't like them.  AFAIK sending/returning an int is fastest and everything else incurs extra conversion.</div>

</div><br>-- <br>Robert Špalek <<a href="mailto:rspalek@gmail.com">rspalek@gmail.com</a>><br>