[x264-devel] Patch open gop
Jason Garrett-Glaser
darkshikari at gmail.com
Thu Sep 3 06:35:41 CEST 2009
Trahald,
Here's my current local patch; I've revamped the commandline usage
once again and cleaned it up a bit. Here's Loren's requests before we
commit it (why aren't you paying attention in IRC--it would get done a
lot faster that way?):
12:18 < pengvado> I think x264_reference_partial_reset could be simpler if it
processes B-frames rather than skipping non-B-frames
12:20 < pengvado> it should also do if(!pyramid) return at the beginning,
rather than while(pyramid) at the end.
12:22 < pengvado> ref list header code should be macroed, functioned, or looped
for the 2 lists, not duplicated
12:23 < pengvado> ref list reordering should be enabled only if the desired
order actually differs from the default order
12:24 < pengvado> which should never happen in B-frames
12:24 < pengvado> (unless you're abusing then for forward-only prediction or
something like that)
12:28 < pengvado> why do you need a mmco[] in both slice header and x264_t?
12:29 < pengvado> if it's because x264_reference_partial_reset runs before
slice_header_init, then just don't overwrite it in
slice_header_init
12:33 < pengvado> x264_reference_partial_reset should be named something that
says which part, such as x264_reference_hierarchy_reset
12:34 < pengvado> and then there's the comment on reordering, but I already
sent that
12:34 < pengvado> nothing else
Dark Shikari
-------------- next part --------------
A non-text attachment was scrubbed...
Name: test.diff
Type: application/octet-stream
Size: 16440 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/x264-devel/attachments/20090902/507d68d6/attachment.obj>
More information about the x264-devel
mailing list