[Fontconfig] fontconfig: Branch 'master'

Keith Packard keithp at kemper.freedesktop.org
Wed Dec 12 21:48:27 PST 2007


 src/fccfg.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0f7870887adff6db3cffda5485418143f2bfa1f6
Author: Hongbo Zhao <hongbo.zhao at gmail.com>
Date:   Wed Dec 12 21:47:33 2007 -0800

    Not_contain should use strstr, not strcmp on strings. (bug 13632)
    
    For Version 2.5.0, (same for previous version 2.4.2), in source file fccfg.c,
    on line 700,
    
    Original:
          ret = FcStrCmpIgnoreCase (left.u.s, right.u.s) == 0;
    
    Should change to:
          ret = FcStrStrIgnoreCase (left.u.s, right.u.s) == 0;
    
    I think this is just a mistake when copy-n-paste similar codes in the same
    function. Apparently, return for "Not_contain" should be just the inverse of
    "Contain", not the same as "Equal".

diff --git a/src/fccfg.c b/src/fccfg.c
index c73dc2a..ed620a0 100644
--- a/src/fccfg.c
+++ b/src/fccfg.c
@@ -697,7 +697,7 @@ FcConfigCompareValue (const FcValue	*left_o,
 		ret = FcStrCmpIgnoreCase (left.u.s, right.u.s) != 0;
 		break;
 	    case FcOpNotContains:
-		ret = FcStrCmpIgnoreCase (left.u.s, right.u.s) == 0;
+		ret = FcStrStrIgnoreCase (left.u.s, right.u.s) == 0;
 		break;
 	    default:
 		break;


More information about the Fontconfig mailing list