Hi Radek,<div><br><div class="gmail_quote">On Sat, Jun 25, 2011 at 3:47 AM, Radek Doulik <span dir="ltr">&lt;<a href="mailto:rodo@novell.com">rodo@novell.com</a>&gt;</span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I run into a crash on startup in that part. I quickly fixed it, but it<br>
might be better if you check that it plays nicely with your new code. Or<br>
maybe make the ScrollToAbsPos not crash when called with unexpected<br>
values, -1 for example.</blockquote><div><br></div><div>Thanks for fixing this! You&#39;re right, I didn&#39;t test loading a blank presentation... Your quick fix fits well with the rest of the code; nFirstVis will never be -1 if there are entries in the CustomAnimationList. I was sad that ScrollToAbsPos failed; it turns out that it&#39;s due to there being no entries in the list (I had added a check for negative scroll position values which would try to go to scroll to the first entry). I have attached a patch for ScrollToAbsPos that will check if there are entries in the list first, and if not just return. Could you please review/commit this?</div>
<div><br></div><div>Kind regards,</div><div><br></div><div>Luke.</div><meta http-equiv="content-type" content="text/html; charset=utf-8"></div></div>