[FFmpeg-devel] [PATCH 2/3] avcodec/codec_par: Move enum AVFieldOrder to defs.h

[Anton Khirnov]

Quoting Andreas Rheinhardt (2023-09-26 11:17:25)
> James Almer:
> > On 9/25/2023 8:55 PM, Andreas Rheinhardt wrote:
> >> It is also used by AVCodecContext.
> >>
> >> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> >> ---
> >>   doc/APIchanges         |  3 +++
> >>   libavcodec/codec_par.h | 10 +---------
> >>   libavcodec/defs.h      |  8 ++++++++
> >>   3 files changed, 12 insertions(+), 9 deletions(-)
> >>
> >> diff --git a/doc/APIchanges b/doc/APIchanges
> >> index ca7ffbb97e..c1ea3b7dc1 100644
> >> --- a/doc/APIchanges
> >> +++ b/doc/APIchanges
> >> @@ -2,6 +2,9 @@ The last version increases of all libraries were on
> >> 2023-02-09
> >>     API changes, most recent first:
> >>   +2023-09-26 - xxxxxxxxxx - lavc 60.xx.100 - codec_par.h defs.h
> >> +  Move the definition of enum AVFieldOrder from codec_par.h to defs.h.
> > 
> > Does this need an entry? If codec_par.h includes defs.h, it's basically
> > transparent for the API user.
> > 
> 
> It does not break existing users, but existing users may take advantage
> of it. Probably not in this case, but for moving stuff in general and
> therefore we add these APIchanges entries.
> But if you insist, I can omit it.

I think it should be mentioned.

-- 
Anton Khirnov