[FFmpeg-devel] [PATCH v3 1/2] lavf: Add general API for IO buffer synchronization.

Michael Niedermayer michael at niedermayer.cc
Fri Dec 21 01:48:00 EET 2018

On Thu, Dec 20, 2018 at 11:00:32AM +0100, Nicolas George wrote:
> Andrey Semashev (2018-12-10):
> > This commit adds a new set of functions to avio and url subsystems, which
> > allow users to invoke IO buffer synchronization with the underlying media.
> > The most obvious target for this extension if the filesystem streams. Invoking
> > IO synchronization allows user applications to ensure that all written content
> > has reached the filesystem on the media and can be observed by other processes.
> > 
> > The public API for this is avio_sync() function, which can be called on
> > AVIOContext. The internal, lower layer API is ffurl_sync(), which works
> > directly on the underlying URLContext. URLContext backends can add support for
> > this new API by setting the sync handler to the new url_sync member of
> > URLProtocol. When no handler is set then the sync operation is a no-op,
> > the result code is AVERROR(ENOSYS).
> > ---
> >  libavformat/avio.c    |  7 +++++++
> >  libavformat/avio.h    | 20 ++++++++++++++++++++
> >  libavformat/aviobuf.c | 17 +++++++++++++++++
> >  libavformat/url.h     | 12 ++++++++++++
> >  4 files changed, 56 insertions(+)
> I have no more objections to the patch as it is, thanks. But I have no
> opinion in favor of it either; it is useful in principle, but I do not
> know if it is worth the extra maintenance for a use case like this
> project. I will leave the judgement to others.

IMHO, it looks useful and extra maintaince needed seems not major



Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20181221/0778414f/attachment.sig>

More information about the ffmpeg-devel mailing list