[FFmpeg-devel] [PATCH 1/5] avutil/tx_template: extend to 2M

[Paul B Mahol]

On Mon, Jul 24, 2023 at 10:34 AM Andreas Rheinhardt <
andreas.rheinhardt at outlook.com> wrote:

> Michael Niedermayer:
> > On Sat, Jun 17, 2023 at 12:20:44AM +0200, Michael Niedermayer wrote:
> >> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> >> ---
> >>  libavutil/tx_template.c | 12 ++++++++++++
> >>  1 file changed, 12 insertions(+)
> >
> > will apply patches 1-4
> > (this should reduce the differences in the avradio repository in
> libavcodec/libavutil)
> >
>
> I actually wanted to complain about this; but then I saw that it had
> already been applied. Before that, I believed that this patchset was
> meant to be applied to the fork only and therefore didn't give it any
> scrutiny. These patches should have only been applied after the decision
> to merge the fork.
>
> I am against applying patches to simplify maintaining a fork. E.g. I am
> against further avpriv functions (but not absolutely) like you added in
> 4/5. Your patches are even very suboptimal in this regard: You added two
> avpriv functions, yet only one of them (the non-fixed-point) is used by
> your fork. Furthermore, you only need these 2MB windows for floats, not
> for int or doubles.
>
>
I need this, and only this patch to stay in tree.


> - Andreas
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>