src/fccfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) New commits: commit 0f7870887adff6db3cffda5485418143f2bfa1f6 Author: Hongbo Zhao <hongbo.zhao@xxxxxxxxx> 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; _______________________________________________ Fontconfig mailing list Fontconfig@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/fontconfig