On Mon, 19 Aug 2013, John Stebbins wrote:
On 08/19/2013 01:24 PM, Luca Barbato wrote:
On 19/08/13 20:08, John Stebbins wrote:
From: John Stebbins <[email protected]>
Adds "moov_size" option to reserve space for moov at the front of the
file.
^
Probably we should put in the documentation which value is sane and/or
provide a guess if the user does not know better.
I considered *not* merging this patch for this reason.HandBrake doesn't
actually need this. But since I needed the
faststart patch, I thought I would go ahead and bring this related patch along
with it. We could drop this patch if you
think it just leads to trouble (I would have to rework the faststart patch
some).
I don't mind that much either way. It's a pretty niche feature that is
really hard to use correctly (and probably is mostly of historical
interest if the faststart option exists), so perhaps it'd be easier to
just squash the two patches and remove what isn't necessary for the
faststart mode? If you find the feature useful we can of course keep it as
well (even if you won't use it in handbrake).
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
