[Fontconfig] fontconfig: Branch 'master' - 9 commits

Alexey Khoroshilov khoroshilov at ispras.ru
Sun Nov 4 10:38:04 PST 2007



Keith Packard wrote:
>     Typo error in function name: Inverval -> interval
>
> diff --git a/src/fccfg.c b/src/fccfg.c
> index b55770d..18a74e7 100644
> --- a/src/fccfg.c
> +++ b/src/fccfg.c
> @@ -508,7 +508,7 @@ FcConfigAddBlank (FcConfig	*config,
>  }
>  
>  int
> -FcConfigGetRescanInverval (FcConfig *config)
> +FcConfigGetRescanInterval (FcConfig *config)
>  {
>      if (!config)
>      {
> @@ -520,7 +520,7 @@ FcConfigGetRescanInverval (FcConfig *config)
>  }
>  
>  FcBool
> -FcConfigSetRescanInverval (FcConfig *config, int rescanInterval)
> +FcConfigSetRescanInterval (FcConfig *config, int rescanInterval)
>  {
>      if (!config)
>      {
This fix changes the ABI of fonconfig essentially.
May be it would be useful to smooth down the change by providing the old 
binary interface as well as the new one.
Otherwise some applications will not be able to work on systems with old 
and new fontconfig libraries simultaneously.

For example we can use gcc-specific alias attribute:

#define define_alias(name, aliasname) \
  extern __typeof(name) aliasname __attribute__((alias (#name)));

/*
 * Define FcConfigSetRescanInverval as an alias to FcConfigSetRescanInterval
 * to enable backward compatibility of fontconfig ABI.
 */
define_alias( FcConfigSetRescanInterval, FcConfigSetRescanInverval )


What do you think?

--
Alexey



More information about the Fontconfig mailing list