> 
>>> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
>>> index 0ce22ec4fa..7087f82ce1 100644
>>> --- a/libavcodec/avcodec.h
>>> +++ b/libavcodec/avcodec.h
>>> @@ -3357,6 +3357,12 @@ typedef struct AVCodecContext {
>>>     * - encoding: unused
>>>     */
>>>    int discard_damaged_percentage;
>>> +
>>> +    /*
>>> +     * Video encoding only. Sets the maximum picture size for encoders that
>>> +     * support adjusting the picture size dynamically during encoding.
>>> +     */
>>> +     int max_width, max_height;
>>> } AVCodecContext;
>>> 
>> 
>> I really don't like introducing public API fields for this. Maybe a
>> private nvenc option would be better at this point.
>> 
> 
> So would this be suitable?
> 
> The only thing was that I wasn't sure about exposing these options to ffmpeg 
> though, as they're useless to ffmpeg unless ffmpeg were to add features for 
> dynamically changing the encoded resolution....I just couldn't figure out how 
> you can add a private option that isn't exposed in this way though.
> 

So in the absence of any further feedback I've created a patch.

Attachment: 0001-nvenc-added-support-for-dynamic-resolution-encoding.patch
Description: Binary data

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to