[FFmpeg-devel] [PATCH] H.264: Shrink unnecessarily large tables

[Jason Garrett-Glaser]

On Mon, Dec 22, 2008 at 4:13 AM, Jason Garrett-Glaser
<darkshikari at gmail.com> wrote:
> On Mon, Dec 22, 2008 at 4:11 AM, Marc Mason <mpeg.blue at free.fr> wrote:
>> Jason Garrett-Glaser wrote:
>>
>>> These tables were considerably larger than necessary because they were
>>> inserted before it was known that the standard had limits on
>>> alpha/beta values.
>>>
>>> This should save about 480 bytes of binary size.
>>
>> Wouldn't this be an opportunity to replace some magic constant with an
>> appropriately named macro ?
>
> Want to do it the way x264 does it then?
>
> #define alpha_table(x) i_alpha_table[(x)+12]
> #define beta_table(x)  i_beta_table[(x)+12]
> #define tc0_table(x)   i_tc0_table[(x)+12]
>
> Dark Shikari
>

Patch attached.

Dark Shikari
-------------- next part --------------
A non-text attachment was scrubbed...
Name: remove_unnecessary_tables.diff
Type: text/x-diff
Size: 10064 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081222/513d1c59/attachment.diff>