[FFmpeg-devel] [PATCH] support for hardcoded mpegaudiodec tables

Reimar Döffinger Reimar.Doeffinger
Tue Oct 27 17:57:00 CET 2009


On Tue, Oct 27, 2009 at 05:29:34PM +0100, Reimar D?ffinger wrote:
> On Tue, Oct 27, 2009 at 05:05:51PM +0100, Michael Niedermayer wrote:
> > up to these your implementation did not look that bad but this here
> > would need 2(signedness)(8/16/32/64/float/double)(1d/2d) ->
> > 24 functions
> 
> Do you mind the number of functions or the code duplication?
> The functions I think can be easily generated by two macros with about
> 4 parameters, one for 1d and one for 2d, getting rid of duplicated code
> and binary size should really not matter here.

More specifically, see attached patch.
I will note that I have been too lazy to implement line-breaking
support for 2d arrays, that can be done if ever it becomes useful...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpatables2.diff
Type: text/x-diff
Size: 10938 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091027/260fc94b/attachment.diff>



More information about the ffmpeg-devel mailing list