[FFmpeg-devel] [PATCH v2 5/5] fftools/ffmpeg: support applying container level cropping

[Anton Khirnov]

Quoting Tomas Härdin (2023-07-26)
> tis 2023-07-25 klockan 14:09 -0300 skrev James Almer:
> > Signed-off-by: James Almer <jamrial at gmail.com>
> > ---
> > Now inserting a filter into the graph.
> 
> This looks useful for MXF
> 
> > +    { "apply_cropping",   HAS_ARG | OPT_BOOL | OPT_SPEC |
> > +                          OPT_EXPERT |
> > OPT_INPUT,                                { .off =
> > OFFSET(apply_cropping) },
> > +        "Apply frame cropping instead of exporting it" },
> 
> Hm. Can this be applied automatically for ffplay? When transcoding I
> expect the typical use case is to not crop and to carry the metadata
> over.

Why? We do apply decoder cropping by default. There's also no guarantee
that your container will be able to write it, so it seems better to
apply it by default.

-- 
Anton Khirnov