[PATCH xfs 2/2] Delete unused CopyISOLatin1Lowered function

Alan Coopersmith alan.coopersmith at oracle.com
Thu May 8 14:31:32 PDT 2014


I think it may have been needed for platforms without weak symbols before
libXfont commit daa7af2bb2326de363, but that's in 1.4.0 and we already
require newer, so seems fine to drop now.   (Why that was ever weak and
overridable in the first place, I can't imagine.  It's not like different
programs have different definitions of ISO-8859-1 encoding.)

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

On 05/ 8/14 10:28 AM, Julien Cristau wrote:
> ---
>   difs/difsutils.c | 20 --------------------
>   1 file changed, 20 deletions(-)
>
> diff --git a/difs/difsutils.c b/difs/difsutils.c
> index 0dd99b5..7e350c6 100644
> --- a/difs/difsutils.c
> +++ b/difs/difsutils.c
> @@ -189,26 +189,6 @@ XpClientIsPrintClient(ClientPtr client, FontPathElementPtr fpe)
>       return FALSE;
>   }
>
> -void
> -CopyISOLatin1Lowered(char *d, const char *s, int length)
> -{
> -    register int i;
> -    unsigned char *dest = (unsigned char *)d;
> -    const unsigned char *source = (const unsigned char *)s;
> -
> -    for (i = 0; i < length; i++, source++, dest++) {
> -	if ((*source >= XK_A) && (*source <= XK_Z))
> -	    *dest = *source + (XK_a - XK_A);
> -	else if ((*source >= XK_Agrave) && (*source <= XK_Odiaeresis))
> -	    *dest = *source + (XK_agrave - XK_Agrave);
> -	else if ((*source >= XK_Ooblique) && (*source <= XK_Thorn))
> -	    *dest = *source + (XK_oslash - XK_Ooblique);
> -	else
> -	    *dest = *source;
> -    }
> -    *dest = '\0';
> -}
> -
>   int
>   strncmpnocase(
>       const char *first,
>


-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list