[FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface

[softworkz .]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of softworkz .
> Sent: Dienstag, 29. April 2025 01:24
> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface
> 
> 
> 
> > -----Original Message-----
> > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Stefano
> > Sabatini
> > Sent: Dienstag, 29. April 2025 00:27
> > To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> > Subject: Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface
> >
> > On date Sunday 2025-04-27 17:54:21 +0000, softworkz . wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> > > > Stefano Sabatini
> > > > Sent: Sonntag, 27. April 2025 12:42
> > > > To: FFmpeg development discussions and patches <ffmpeg-
> > > > devel at ffmpeg.org>
> > > > Subject: Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface
> > > >
> 
> [..]


Hello Stefano,

I have five new commits for this:

fftools/textformat: Rename variables wctx to tctx
fftools/textformat: Cleanup unneeded includes
fftools/textformat: Add validation for TextFormat API
fftools/textformat: Add validation for AVTextWriter implementations
fftools/textformat: Add validation for AVTextFormatter implementations

Yet I don't believe it makes sense to squash them once again back into 
commits that you have reviewed already, they are much easier to review
separately.

So, if you would agree, I'd merge the current patchset first (once 
Michael confirms the zlib issue being resolved) and send the new commits
as a new patchset then?

Thanks,
sw