On Sat, Nov 28, 2015 at 3:19 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> wrote: > On 11/28/2015 7:51 PM, Ganesh Ajjanagadde wrote: >> In principle of course, e.g with more ifdefry, configure, or something >> of that sort. I do not believe this is what he meant. > > I will await his reply to clarify, then. Perhaps do not write so > matter-of-factly. > >> In any case, the point is moot - the implementations are not broken - >> if they are, avutil/libm needs fixing as well. > > Now, yes. It would certainly be nice if this was noted somewhere in the patch > notes > (not necessarily in the commit messages). There are quite a few threads on > the list > from you now regarding tablegen, and it can be a pain in the butt / tedious > to figure > out how the current version differs from the N others.
I assumed from the cover letter that this supersedes all such prior stuff. Anyway, will attempt but can't guarantee improvements in future. > >>> Anyway, shouldn't this patce dropped, since >>> 79abf2d0ded860acf505de22c4f7a750e5e98446 >>> removed hardcoded tables anyway? >> >> No, if you actually read even the first line of the commit message, >> you can see it was only for aac, not for this patch series. > > Indeed, I misread, and missed the 'aac_'. You could certainly be less of > an ass about it, though. Sorry about that. Please do continue to call me out on such things; I do not like unpleasantness on ffmpeg-devel. > > - Derek > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel