[FFmpeg-devel] [PATCH] avformat: make AVStream.pts_wrap_bits public

[Michael Niedermayer]

On Wed, Jun 09, 2021 at 03:07:41PM -0300, James Almer wrote:
> It can be useful to library users, and is currently being used by ffmpeg.c
> 
> Suggested-by: Hendrik Leppkes <h.leppkes at gmail.com>
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  doc/APIchanges         |  3 +++
>  libavformat/avformat.h | 17 +++++++----------
>  libavformat/version.h  |  4 ++--
>  3 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/doc/APIchanges b/doc/APIchanges
> index c46f4d5304..1b25bddd43 100644
> --- a/doc/APIchanges
> +++ b/doc/APIchanges
> @@ -14,6 +14,9 @@ libavutil:     2021-04-27
>  
>  API changes, most recent first:
>  
> +2021-06-09 - xxxxxxxxxx - lavf 59.3.100 - avformat.h
> +  Add pts_wrap_bits to AVStream
> +
>  2021-04-27 - cb3ac722f4 - lavc 59.0.100 - avcodec.h
>    Constified AVCodecParserContext.parser.
>  

> diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> index 094683f12a..0d12d5b0d2 100644
> --- a/libavformat/avformat.h
> +++ b/libavformat/avformat.h
> @@ -980,17 +980,14 @@ typedef struct AVStream {
>       */
>      AVCodecParameters *codecpar;
>  
> -    /*****************************************************************
> -     * All fields below this line are not part of the public API. They
> -     * may not be used outside of libavformat and can be changed and
> -     * removed at will.
> -     * Internal note: be aware that physically removing these fields
> -     * will break ABI. Replace removed fields with dummy fields, and
> -     * add new fields to AVStreamInternal.
> -     *****************************************************************
> +    /**
> +     * Number of bits in pts. Used for wrapping control.

pts and dts or in timestamps, its not just pts (no need to fix it in this patch
as its just copied as i realize) but it should be fixed when its made public

patch LGTM

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210610/9aeec5d8/attachment.sig>