[FFmpeg-devel] [PATCH 1/2] lavf: add audio_preload option, this allows interleaving audio earlier
Michael Niedermayer
michaelni at gmx.at
Wed Nov 30 05:34:46 CET 2011
On Tue, Nov 29, 2011 at 02:49:12PM -0900, Lou Logan wrote:
> On Wed, 30 Nov 2011 00:29:13 +0100, Michael Niedermayer wrote:
>
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> > libavformat/avformat.h | 7 +++++++
> > libavformat/options.c | 1 +
> > libavformat/utils.c | 10 ++++++++++
> > 3 files changed, 18 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> > index 752645e..b281406 100644
> > --- a/libavformat/avformat.h
> > +++ b/libavformat/avformat.h
> > @@ -1078,6 +1078,13 @@ typedef struct AVFormatContext {
> > */
> > int ts_id;
> >
> > + /**
> > + * Audio preload in micro seconds.
>
> I'd prefer microseconds (to be consistent with the usage in audio_preload).
>
> > + * Note, not all formats support this and unpredictable things may happen if its used when not supported.
>
> it's or it is
>
> Same with patch 2/2. Incredibly small nits only to be addressed if you
> are making further changes.
fixed together with a few other bigger issues
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111130/b37ff9f5/attachment.asc>
More information about the ffmpeg-devel
mailing list