[FFmpeg-devel] [PATCH] avcodec: Extend fft to size 2^17

[Ronald S. Bultje]

Hi,

On Fri, Mar 4, 2016 at 7:52 AM, Paul B Mahol <onemda at gmail.com> wrote:

> On 3/4/16, wm4 <nfxjfg at googlemail.com> wrote:
> > On Fri, 4 Mar 2016 13:12:45 +0100
> > Paul B Mahol <onemda at gmail.com> wrote:
> >
> >> On 3/4/16, Michael Niedermayer <michael at niedermayer.cc> wrote:
> >> > On Fri, Mar 04, 2016 at 11:21:23AM +0100, Paul B Mahol wrote:
> >> >> On 3/4/16, Michael Niedermayer <michael at niedermayer.cc> wrote:
> >> >> > Hi
> >> >> >
> >> >> > patch to extend fft is attached (my git-send email atm doesnt work
> >> >> > thanks to my ISP)
> >> >> >
> >> >> > --
> >> >> > Michael     GnuPG fingerprint:
> >> >> > 9FF2128B147EF6730BADF133611EC787040B0FAB
> >> >> >
> >> >> > I know you won't believe me, but the highest form of Human
> Excellence
> >> >> > is
> >> >> > to question oneself and others. -- Socrates
> >> >> >
> >> >>
> >> >> This crashes whenever I tried to use it with sofalizer/afftfilt:
> >> >
> >> > fixed with af_afftfilt and applied
> >> >
> >> >
> >> > sofalizer just crashes here even wthout any changes:
> >>
> >> That's their awful library.
> >
> > Oh really? Are you sure they allow passing a NULL filename to their
> > open function?
> >
>
> It allows here, but Michael version is older.


configure version check seems like a sane thing to add. Possibly even a
runtime check just to be sure. "ffmpeg crashed" never looks good.

Ronald