<br>
<br><br><div class="gmail_quote">On Mon, Jun 7, 2010 at 5:10 PM, salsaman <span dir="ltr"><<a href="mailto:salsaman@gmail.com">salsaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br><br><div class="gmail_quote"><div class="im">On Mon, Jun 7, 2010 at 4:42 PM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">


<div>On Mon, Jun 07, 2010 at 08:59:08AM -0300, salsaman wrote :<br>
</div>> diff --git a/modules/codec/theora.c b/modules/codec/theora.c<br>
<br>
> +    /* need to skip some pages after a seek */<br>
<div>> +    if ( p_block->i_flags & BLOCK_FLAG_PREROLL )<br>
> +        {<br>
> +            p_sys->b_skip_frame = true;<br>
> +    }<br>
</div>> +    else p_sys->b_skip_frame = false;<br>
Wouldn't something like this, be better?<br>
p_sys->b_skip_frame = ( p_block->i_flags & BLOCK_FLAG_PREROLL );<br>
<br>
<br></blockquote></div><div><br></div></div></blockquote><div><br><br><br><br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="gmail_quote">

<div><br>Yes, you are right. I will change it.<br><br></div></div></blockquote><div><br><br>Actually, come to think of it, no, strictly speaking:<br><br>e.g.<br><br>BLOCK_FLAG_PREROLL = 2<br><br>p_block->i_flags = 2<br>

<br>p_block->i_flags&BLOCK_FLAG_PREROLL = 2;<br><br>true = 1;<br><br>Ooops !!<br><br><br>Gabriel.<br><br></div></div>