ok.. partial reset renamed and slimmed down, ref list header macroed , reordering only done for bref, mmco only defined in sh<br><br>the reordering thing i dont plan to do ( im slow would take me too long, and really dont know where to start, needs opengop ) <br>
<div class="gmail_quote">On Thu, Sep 3, 2009 at 2:37 PM, Trahald <span dir="ltr">&lt;<a href="mailto:wewk584@gmail.com">wewk584@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
normal mode needs reordering for correct output. (actually just for the BREFs.. i can change it to x264_type_bref instead of slicetypeb.).<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Thu, Sep 3, 2009 at 12:35 AM, Jason Garrett-Glaser <span dir="ltr">&lt;<a href="mailto:darkshikari@gmail.com" target="_blank">darkshikari@gmail.com</a>&gt;</span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="h5">Trahald,<br>
<br>
Here&#39;s my current local patch; I&#39;ve revamped the commandline usage<br>
once again and cleaned it up a bit.  Here&#39;s Loren&#39;s requests before we<br>
commit it (why aren&#39;t you paying attention in IRC--it would get done a<br>
lot faster that way?):<br>
<br>
12:18 &lt; pengvado&gt; I think x264_reference_partial_reset could be simpler if it<br>
                  processes B-frames rather than skipping non-B-frames<br>
12:20 &lt; pengvado&gt; it should also do if(!pyramid) return at the beginning,<br>
                  rather than while(pyramid) at the end.<br>
12:22 &lt; pengvado&gt; ref list header code should be macroed, functioned, or looped<br>
                  for the 2 lists, not duplicated<br>
12:23 &lt; pengvado&gt; ref list reordering should be enabled only if the desired<br>
                  order actually differs from the default order<br>
12:24 &lt; pengvado&gt; which should never happen in B-frames<br>
12:24 &lt; pengvado&gt; (unless you&#39;re abusing then for forward-only prediction or<br>
                  something like that)<br>
12:28 &lt; pengvado&gt; why do you need a mmco[] in both slice header and x264_t?<br>
12:29 &lt; pengvado&gt; if it&#39;s because x264_reference_partial_reset runs before<br>
                  slice_header_init, then just don&#39;t overwrite it in<br>
                  slice_header_init<br>
12:33 &lt; pengvado&gt; x264_reference_partial_reset should be named something that<br>
                  says which part, such as x264_reference_hierarchy_reset<br>
12:34 &lt; pengvado&gt; and then there&#39;s the comment on reordering, but I already<br>
                  sent that<br>
12:34 &lt; pengvado&gt; nothing else<br>
<br>
Dark Shikari<br>
<br></div></div><div class="im">_______________________________________________<br>
x264-devel mailing list<br>
<a href="mailto:x264-devel@videolan.org" target="_blank">x264-devel@videolan.org</a><br>
<a href="http://mailman.videolan.org/listinfo/x264-devel" target="_blank">http://mailman.videolan.org/listinfo/x264-devel</a><br>
<br></div></blockquote></div><br>
</blockquote></div><br>