[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