diff options
author | Ben Wagner <bungeman@chromium.org> | 2020-12-04 15:42:28 -0500 |
---|---|---|
committer | Akira TAGOH <akira@tagoh.org> | 2020-12-07 04:21:38 +0000 |
commit | 97d541855429629f40a0accdd2b06b7dce7ba2db (patch) | |
tree | 68263556530a1ec4d54659bc7318ac0387f7b567 | |
parent | 3d6926380dc3c8597dc2fd9d34087da9b39dfdd9 (diff) |
Remove abort from FcCompareSize.
There doesn't appear to be a good reason to abort when 'v1' has type
FcTypeRange. If there does turn out to be a good reason for this then it
should be better documented and the code for handling this case removed.
At worst it seems -1 should be returned as it is for other unknown
types. It is possible this is left over debug code from the initial
implementation.
-rw-r--r-- | src/fcmatch.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/fcmatch.c b/src/fcmatch.c index df6db71..53c3b0e 100644 --- a/src/fcmatch.c +++ b/src/fcmatch.c @@ -235,7 +235,6 @@ FcCompareSize (const FcValue *v1, const FcValue *v2, FcValue *bestValue) b1 = e1 = value1.u.d; break; case FcTypeRange: - abort(); b1 = value1.u.r->begin; e1 = value1.u.r->end; break; |