[FFmpeg-devel] [PATCH, v2 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag
Anton Khirnov
anton at khirnov.net
Tue Feb 18 15:32:22 EET 2020
Quoting Michael Niedermayer (2020-02-17 23:38:31)
> On Mon, Feb 17, 2020 at 06:28:23PM +0000, Fu, Linjie wrote:
>
> > > > > https://patchwork.ffmpeg.org/patch/14122/
>
> iam hesitating because it feeds encoders with changing resolution resulting
> in potentially undefined behavior ...
>
>
> > > > > https://patchwork.ffmpeg.org/patch/14139/
>
> long discussion here its not immedeatly clear if anyone was against it
>
> Also there is the question about the API, is there anything in the API
> documentation that restricts the user app from changing the encoder
> input frame dimensions?
> This should be documented somewhere if its not ...
>
> If a flag is added that affects this, it would have to
> be documented so someone writing a user app using the encoders
> would know if they are allowed to change the resolution.
> With just the flag and its documentation a developer could miss
> the flag entirely
Is there even a sufficiently strong use case for this? Why not just
create a new encoder instance?
--
Anton Khirnov
More information about the ffmpeg-devel
mailing list