commit f034c86756d45bed61b86310d9e4e77db2d05df3
author Behdad Esfahbod <behdad@behdad.org> Wed Sep 13 03:29:20 2017 -0400
committer Behdad Esfahbod <behdad@behdad.org> Wed Sep 20 13:13:35 2017 -0700
tree fe48867bc9a9be080b5b0efa25ce9e678267d0af
parent a4bd5b7c7a06fe39d1461f9be098af37d364dcc2

Print ranges as closed as opposed to half-open

There's nothing assymetrical about how we match them.  Previously we "considered"
them half-open because the OS/2 spec had usLowerOpticalPointSize as inclusive
and usUpperOpticalPointSize as exclusive.  But we do not respect that.

Note that the parsing code accepts both anyway, because of the way our sscanf()
usage is written...

src/fcdbg.c

diff --git a/src/fcdbg.c b/src/fcdbg.c
index c2853ff..40e0d66 100644
--- a/src/fcdbg.c
+++ b/src/fcdbg.c
@@ -62,7 +62,7 @@
 	fprintf (f, "face");
 	break;
     case FcTypeRange:
-	fprintf (f, "[%g %g)", v.u.r->begin, v.u.r->end);
+	fprintf (f, "[%g %g]", v.u.r->begin, v.u.r->end);
 	break;
     }
 }

src/fcname.c

diff --git a/src/fcname.c b/src/fcname.c
index 8be36c7..77e74bc 100644
--- a/src/fcname.c
+++ b/src/fcname.c
@@ -318,7 +318,7 @@
 	    v.type = FcTypeVoid;
 	break;
     case FcTypeRange:
-	if (sscanf ((char *) string, "[%lg %lg)", &b, &e) != 2)
+	if (sscanf ((char *) string, "[%lg %lg]", &b, &e) != 2)
 	{
 	    v.u.d = strtod ((char *) string, &p);
 	    if (p != NULL && p[0] != 0)
@@ -526,7 +526,7 @@
     case FcTypeFTFace:
 	return FcTrue;
     case FcTypeRange:
-	sprintf ((char *) temp, "[%g %g)", v.u.r->begin, v.u.r->end);
+	sprintf ((char *) temp, "[%g %g]", v.u.r->begin, v.u.r->end);
 	return FcNameUnparseString (buf, temp, 0);
     }
     return FcFalse;