[FFmpeg-devel] [PATCH 1/2] avcodec/s302m: enable non-PCM decoding

[Anton Khirnov]

Quoting Gyan Doshi (2024-02-15 17:47:49)
> This patch facilitates a certain productive use of ffmpeg with respect 
> to processing of live inputs that wasn't possible earlier,
> and which currently is being used successfully by multiple people over 
> the past few weeks.
> It applies a processing model already implemented in multiple other 
> decoders for a number of years. I haven't seen many reports
> of issues with them. And surely something being 'a constant source of 
> issues' would be a lot more than 'subtly broken' as you describe 
> them.

This reads very much like "I can't be bothered to do it properly and
would rather someone else fix it in the future". Given past experience,
that someone is highly likely to be me, and fixing past architectural
decisions requires a lot more effort than doing things properly in the
first place..

> You're the only one who has objected on architectural grounds and 

Not true, Andreas has objected as well.

> If you are blocking this patch, do acknowledge here within 24 hours and 
> we can send this to the TC else I'll push it after that period.

I must say I rather dislike this "my way or the highway" attitude. So
yes, I am objecting to the patch in its current form.

-- 
Anton Khirnov