[FFmpeg-devel] [PATCH] Make postprocess.c's replaceTable fall into .rodata

[Diego 'Flameeyes' Pettenò]

Michael Niedermayer <michaelni at gmx.at> writes:

> the strings used for replacements are not in their shorest form ...
> x1hdeblock == h1 for example
> this would change the distribution of the lengths and maybe make your
> proposed change not need more mem

You were right, I didn't look at that before. The attached patch reduces
the replacements with their short form, so an entry would be at most 21
bytes, I've used 24 so that 8 + 24 = 32 bytes would be a power of two.

The table should still be using more memory on 32-bit systems (but not
so much), but it should be smaller on 64-bit systems.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: libpostproc-rodata-replaceTable.patch
Type: text/x-patch
Size: 2429 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080108/be56c6ac/attachment.bin>
-------------- next part --------------


-- 
Diego "Flameeyes" Petten?
http://farragut.flameeyes.is-a-geek.org/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080108/be56c6ac/attachment.pgp>