Re: fontconfig: Branch 'master'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/ 7/12 02:10 AM, tagoh@xxxxxxxxxxxxxxxxxxxxxx wrote:
> @@ -81,12 +81,15 @@ FcInitLoadConfig (void)
>  		 "Fontconfig warning: adding <cachedir>%s</cachedir>\n",
>  		 FC_CACHEDIR);
>  	prefix = FcConfigXdgCacheHome ();
> -	plen = prefix ? strlen ((const char *)prefix) : 0;
>  	if (!prefix)
>  	    goto bail;
> -	prefix = realloc (prefix, plen + 12);
> -	if (!prefix)
> +	plen = prefix ? strlen ((const char *)prefix) : 0;

Since you moved that line to after the "if (!prefix) goto bail",
can't you get rid of the now redundant check for a null prefix
and simply make that:
	plen = strlen((const char *)prefix):
now?


-- 
	-Alan Coopersmith-              alan.coopersmith@xxxxxxxxxx
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
Fontconfig mailing list
Fontconfig@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/fontconfig


[Index of Archives]     [Fedora Fonts]     [Fedora Users]     [Fedora Cloud]     [Kernel]     [Fedora Packaging]     [Fedora Desktop]     [PAM]     [Gimp Graphics Editor]     [Yosemite News]

  Powered by Linux