[PATCH 1/2 v2] Use one single function to register fpe functions

Vignatti Tiago (Nokia-D/Helsinki) tiago.vignatti at nokia.com
Fri Jun 11 06:07:56 PDT 2010


On Fri, Jun 11, 2010 at 02:35:23PM +0200, Vignatti Tiago (Nokia-D/Helsinki) wrote:
> X server doesn't need to understand fpe internals, so let it transparent
> turning all registration functions in a single one. For that, fill the already
> existent register_fpe_functions().
> 
> Some X servers don't want font server support, so this patch also sets font
> server support to be configured in build time.
> 
> In my machine, I see 20kB of RSS being saved in libXfont mapped in Xorg
> process when I disabled font server support and other kind of fonts in the
> library (--disable-pcfformat --disable-bdfformat --disable-snfformat
> --disable-freetype --disable-fc).
> 
> The default library built was taking:
>    text    data     bss     dec     hex filename
>  261847    4484    1536  267867   4165b ./lib/libXfont.so
> 
> and with these flags, it jumps to:
>    text    data     bss     dec     hex filename
>  157764    2428    1188  161380   27664 ./lib/libXfont.so
> 
> Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>

argh. I forgot to stamp here the reviews:
Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>
Reviewed-by: Alex Deucher <alexdeucher at gmail.com>

        Tiago


More information about the xorg-devel mailing list