<div class="gmail_quote">On Tue, Jun 30, 2009 at 3:31 PM,  <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: 41979<br>
          <a href="http://scummvm.svn.sourceforge.net/scummvm/?rev=41979&view=rev" target="_blank">http://scummvm.svn.sourceforge.net/scummvm/?rev=41979&view=rev</a><br>
Author:   dkasak13<br>
Date:     2009-06-30 22:31:29 +0000 (Tue, 30 Jun 2009)<br>
<br>
Log Message:<br>
-----------<br>
Added the Drawable abstract base class and made Sprite inherit from it.<br>
<br>
Modified Paths:<br>
--------------<br>
    scummvm/branches/gsoc2009-draci/engines/draci/sprite.cpp<br>
    scummvm/branches/gsoc2009-draci/engines/draci/sprite.h<br>
<br>
Modified: scummvm/branches/gsoc2009-draci/engines/draci/sprite.cpp<br>
===================================================================<br>
--- scummvm/branches/gsoc2009-draci/engines/draci/sprite.cpp    2009-06-30 22:20:03 UTC (rev 41978)<br>
+++ scummvm/branches/gsoc2009-draci/engines/draci/sprite.cpp    2009-06-30 22:31:29 UTC (rev 41979)<br>
@@ -54,7 +54,11 @@<br>
  *  Constructor for loading sprites from a raw data buffer, one byte per pixel.<br>
  */<br>
 Sprite::Sprite(byte *raw_data, uint16 width, uint16 height, uint16 x, uint16 y,<br>
-                          bool columnwise) : _width(width), _height(height), _x(x), _y(y), _data(NULL) {<br>
+                          bool columnwise) : _data(NULL) {<br>
+        _width = width;<br>
+        _height = height;<br>
+        _x = x;<br>
+        _y = y;</blockquote><div><br></div><div>is there a reason you change initialization _x(x) into assignment _x = x?  at the end they have exactly the same effect, but if you already had it one way, why are you changing it now?</div>

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