FcObjectValidType: tweak -1 checking

Newer gcc doesn't like when you switch on an enum and use a value
that isn't declared:

fcname.c: In function 'FcObjectValidType':
fcname.c:299:2: warning: case value '4294967295'
	not in enumerated type 'FcType' [-Wswitch]

So tweak the logic to avoid this warning.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

diff --git a/src/fcname.c b/src/fcname.c
index 864ad407df76f5f412e6cbe22b2b49fdc0ecb067..1b32b0ff617b3bb2006d073bfeafab645cdc58c6 100644 (file)
--- a/src/fcname.c
+++ b/src/fcname.c
@@ -296,8 +296,6 @@ FcObjectValidType (FcObject object, FcType type)
 
     if (t) {
        switch (t->type) {
-       case -1:
-           return FcTrue;
        case FcTypeDouble:
        case FcTypeInteger:
            if (type == FcTypeDouble || type == FcTypeInteger)
@@ -308,7 +306,7 @@ FcObjectValidType (FcObject object, FcType type)
                return FcTrue;
            break;
        default:
-           if (type == t->type)
+           if (t->type == -1 || type == t->type)
                return FcTrue;
            break;
        }