On Thu, Mar 24, 2016 at 8:55 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 24/03/16 11:58, Diego Biurrun wrote: >> Noticed one more bit that can be pulled out from various codecs. >> mathtables.c may not be the best place in the world for it, but >> ff_zigzag_direct lives there already. > > If/once it grows bigger that what's plausible it can be split, for now > it seems fine.
I would be leaning towards moving both functions in a separate file (zigzagdata.c?) and adjust dependencies accordingly, in order to reduce the number of code moves (also mathops does seem the wrong place for these tables). -- Vittorio _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel