Lynne: > Jan 12, 2021, 08:50 by andreas.rheinha...@gmail.com: > >> Lynne: >> >>> Jan 9, 2021, 20:22 by d...@lynne.ee: >>> >>>> No longer used by anything. >>>> Unfortunately the old FFT_FLOAT/FFT_FIXED_32 is left as-is. It's >>>> simply too much work for code meant to be all removed anyway. >>>> >>>> Patch attached. Read patch 1/6 to see the size savings. >>>> >>> Forgot to remove the tests, making FATE fail. >>> Fixed, patch attached. >>> >> According to patchwork, even the very first of your patches doesn't pass >> FATE. And given that your second version (of the first patch) didn't >> change anything wrt FATE, it won't be different with v2. >> > > Why are you posting this as a reply to this patch then? > I only tested fate-ac3 and fate-fft. This wasn't covered by either. > Fixed locally. It was a 2-char fix in fate-unknown_layout-ac3. > Because IMO the topic of this mail is "FATE failures", so I answered here.
- Andreas PS: Are you sure you do not need to change fate-lavf-rm, too? _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
