[FFmpeg-devel] [PATCH] avformat/avformat: Remove redundant "NOT PART OF PUBLIC API"
Andreas Rheinhardt
andreas.rheinhardt at gmail.com
Wed Mar 18 16:31:03 EET 2020
AVStream.request_probe as well as AVStream.mux_ts_offset are below the
separator of public and private fields, so that a further "NOT PART OF
PUBLIC API" is redundant.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
I initially also wanted to remove the internal note for AVStream's private
fields ("be aware that physically removing these fields will break
ABI"), but then I became aware that ffmpeg.c uses several of them
(pts_wrap_bits, first_dts, cur_dts, codec_info_nb_frames). If I am not
mistaken, this means that one can't even add new public fields in front
of the public/private boundary in AVStream at all.
libavformat/avformat.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index 9b9b634ec3..f9291a26ec 100644
--- a/libavformat/avformat.h
+++ b/libavformat/avformat.h
@@ -1124,7 +1124,6 @@ typedef struct AVStream {
* -1 -> probing finished
* 0 -> no probing requested
* rest -> perform probing with request_probe being the minimum score to accept.
- * NOT PART OF PUBLIC API
*/
int request_probe;
/**
@@ -1170,7 +1169,6 @@ typedef struct AVStream {
/**
* Timestamp offset added to timestamps before muxing
- * NOT PART OF PUBLIC API
*/
int64_t mux_ts_offset;
--
2.20.1
More information about the ffmpeg-devel
mailing list