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

+Animation *AnimationManager::addText(int id, bool playing) {<br>
+<br>
+       Animation *anim = new Animation(_vm, kIgnoreIndex);<br>
+<br>
+       anim->setID(id);<br>
+       anim->setZ(257);</blockquote><div><br></div><div>add constants for all Z coordinates now set manually (255, 256, 257, ...)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

+void AnimationManager::pauseAnimations() {<br>
+<br>
+       Common::List<Animation *>::iterator it;<br>
+<br>
+       for (it = _animations.begin(); it != _animations.end(); ++it) {<br>
+               if ((*it)->getID() > 0 || (*it)->getID() == kTitleText) {<br>
+                       // Clean up the last frame that was drawn before stopping<br>
+                       (*it)->markDirtyRect(_vm->_screen->getSurface());<br>
+<br>
+                       (*it)->setPaused(true);<br>
+               }<br>
+       }<br>
+}<br>
+<br>
+void AnimationManager::unpauseAnimations() {<br>
</blockquote><div><br></div><div>these methods have the same code except for setting true/false.  maybe unify them and add a parameter denoting which value to set.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Modified: scummvm/branches/gsoc2009-draci/engines/draci/animation.h<br>
===================================================================<br>
--- scummvm/branches/gsoc2009-draci/engines/draci/animation.h   2009-08-17 18:25:51 UTC (rev 43485)<br>
+++ scummvm/branches/gsoc2009-draci/engines/draci/animation.h   2009-08-17 18:47:17 UTC (rev 43486)<br>
@@ -38,7 +38,10 @@<br>
           kWalkingMapOverlay = -2,<br>
           kTitleText = -3,<br>
           kSpeechText = -4,<br>
-          kUnused = -5 };<br>
+          kInventorySprite = -5,<br>
+          kDialogueLinesID = -6,</blockquote><div><br></div><div>add a comment saying that 4 ID's are allocated below this one, or instead of using the number 4 directly, refer to a constant whose value is this very 4</div>

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