Hi,
Le 30/11/2014 13:11, Christophe Gisquet a écrit :
2014-11-30 13:03 GMT+01:00 Michael Niedermayer <[email protected]>:
not really, no,
that was also why i posted a patch for this, i wasnt sure this is
worth the extra table size
No strong opinion here, I don't think the increased memory/potential
speed impact are critical, in particular for this codec. Mostly
matters for embedded stuff I guess, but the table seems already too
big to fit in most L1 data caches anyway.
I wouldn't bother if there is no asking for it. Michael, does this patch
come because of a request you've had?
If not, I don't think it's worth it.
If you still want to apply it, though, don't forget to fix the comment
in libavcodec/pcm_tablegen.h to "65536 entries per table"
--
Ben
_______________________________________________
ffmpeg-devel mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
