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

Anton Khirnov anton at khirnov.net
Tue Jun 25 16:54:21 EEST 2024


Quoting Paul B Mahol (2024-06-25 15:23:53)
> On Tue, Jun 25, 2024 at 3:12 PM Anton Khirnov <anton at khirnov.net> wrote:
> > I expect plenty of broken files from various sources where the user will
> > want to selectively ignore either codec or container cropping.
> > So we could make this an enum option, with values
> > none/all/codec/container, mapping to 0/1/2/3 respectively. That should
> > keep compatibility with current syntax.
> >
> 
> Enum? Why not flags?

For compatibility reasons we want 1 to be 'all'.

The other option is to special-case numeric values.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list