On Tue, 20 Aug 2013, Martin Storsjö wrote:
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).
Oh, and if you want to keep it, please fix the formatting and all that,
which Luca already pointed out.
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
