> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of Anton
> Khirnov
> Sent: Saturday, November 27, 2021 10:04 AM
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v16 08/16] fftools/ffmpeg: Replace
> sub2video with subtitle frame filtering
>
> Quoting Soft Works (2021-11-27 08:18:37)
> > > -----Original Message-----
> > > From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of Andreas
> > > Rheinhardt
> > > Sent: Friday, November 26, 2021 2:02 PM
> > > To: ffmpeg-devel@ffmpeg.org
> > > Subject: Re: [FFmpeg-devel] [PATCH v16 08/16] fftools/ffmpeg: Replace
> > > sub2video with subtitle frame filtering
> >
> > > Furthermore, missing check.
> > > (Maybe ass subtitle based codecs should set AVCodecContext.width and
> > > height based upon this play_res_x/y?
> >
> > Breaks the decoder API.
>
> Why? I'd think width/height are currently unused for subtitles, so you
> can repurpose them without breaking anything.
It breaks the API because every decoder would be required to do this, and
would need to be changed.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
