On Sun, Jul 24, 2016 at 11:43:41PM -0300, James Almer wrote: > On 7/24/2016 8:13 PM, Michael Niedermayer wrote: [...] > > > >> > >> Looking at this function closely now, it could have been added > >> to dnxhddata.h instead. No need for avpriv_ prefix and to > >> forcefully build an object file full of data tables that it > >> doesn't need. > > > > agree, this would be best in this case > > Are you ok with me reverting this commit and moving the function > to the header? The symbol will of course not be removed until next > major bump since it's already in a release. > Alternatively apply "[PATCH] lavf/movenc: add missing dependency > on dnxhddata" but IMO it's better to just move the thing to the > header.
yes, moving to the header is better thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Into a blind darkness they enter who follow after the Ignorance, they as if into a greater darkness enter who devote themselves to the Knowledge alone. -- Isha Upanishad
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
