On Mon, Jun 19, 2017 at 12:02:52AM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint <[email protected]> > --- > libavformat/avio.h | 13 ++++++++++++- > libavformat/aviobuf.c | 7 +++++++ > libavformat/url.h | 1 + > libavformat/version.h | 2 +- > 4 files changed, 21 insertions(+), 2 deletions(-) > > diff --git a/libavformat/avio.h b/libavformat/avio.h > index 844a5723d3..f14b003ba5 100644 > --- a/libavformat/avio.h > +++ b/libavformat/avio.h > @@ -137,7 +137,13 @@ enum AVIODataMarkerType { > * Trailer data, which doesn't contain actual content, but only for > * finalizing the output file. > */ > - AVIO_DATA_MARKER_TRAILER > + AVIO_DATA_MARKER_TRAILER, > + /** > + * A point in the output bytestream where the underlying AVIOContext > might > + * flush the buffer depending on latency or buffering requirements. > Typically > + * means the end of a packet. > + */ > + AVIO_DATA_MARKER_FLUSH_POINT, > };
needs APIChanges entry i think
[...]
> diff --git a/libavformat/url.h b/libavformat/url.h
> index 910f1e00b3..df90333059 100644
> --- a/libavformat/url.h
> +++ b/libavformat/url.h
> @@ -42,6 +42,7 @@ typedef struct URLContext {
> char *filename; /**< specified URL */
> int flags;
> int max_packet_size; /**< if non zero, the stream is packetized
> with this max packet size */
> + int min_packet_size; /**< if non zero, the stream is packetized
> with this min packet size */
> int is_streamed; /**< true if streamed (no seek possible),
> default = false */
> int is_connected;
> AVIOInterruptCB interrupt_callback;
i was hesitating for a moment when i saw this. Is the location, that is
in the middle instead of the end of the struct intended ?
is nothig outside libavformat using url.h ?
(i was thinking libavdevice but it seems not directly include it)
patch LGTM otherwise
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
