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

[softworkz .]

> -----Original Message-----
> From: Stefano Sabatini <stefasab at gmail.com>
> Sent: Donnerstag, 8. Mai 2025 01:31
> To: softworkz . <softworkz at hotmail.com>
> Cc: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [RFC] Shaping the AVTextFormat API Surface
> 
> On date Saturday 2025-05-03 08:55:42 +0000, softworkz . wrote:
> > > -----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
> [...]
> > 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?
> 
> Feel free to merge patches which have been already approved or
> approved with minor nits - in fact this will simplify the task of
> reviewing. Please give some more time to review the other ones not yet
> approved.

I sent out an e-mail yesterday, asking whether anybody would need more time,
and that I'm planning to apply by the end of the week otherwise.
The set is around for three weeks by now and afaik, at least Andreas 
has reviewed the whole set already.
But if anybody needs more time, I'll surely postpone it.

Thanks
sw