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

[Michael Niedermayer]

On Sat, Jan 05, 2008 at 01:58:37PM +0100, Diego 'Flameeyes' Petten? wrote:
> 
> The attached patch changes the way replaceTable is defined, so that
> instead of being an array of pointers (which falls into
> .data.local.rel.ro with PIC), it's a table of character array. This time
> the size is not a power of two, but I think the compiler would probably
> bad it by itself.
> 
> I'm afraid this time it actually does use more memory than the old
> version, but it's probably negligible on 64-bit arches.

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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080105/09605c53/attachment.pgp>