On Sat, Sep 11, 2021 at 8:31 PM Soft Works <softwo...@hotmail.com> wrote: > > > > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of > > Paul B Mahol > > Sent: Saturday, 11 September 2021 12:51 > > To: FFmpeg development discussions and patches <ffmpeg- > > de...@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] [PATCH v3 01/18] avutil/frame: Subtitle > > Filtering - Add AVMediaType property to AVFrame > > > > > @@ -721,6 +728,8 @@ void av_frame_move_ref(AVFrame *dst, AVFrame > > *src); > > > /** > > > * Allocate new buffer(s) for audio or video data. > > > * > > > + * Note: For subtitle data, use av_frame_get_buffer2 > > > > > > > I dislike this approach. It should be consistent with audio & video. > > I agree, it's not nice. I will just change av_frame_get_buffer instead. >
We don't change existing functions like that. We add new API, or deprecate old ones, we don't change it in such an incompatible manner. But before these functions are even needed, lets just see if its even needed to change anything, depending on how subtitles will ultimately end up packaged. - Hendrik _______________________________________________ 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".
