Merge pull request #239 from linebender/misc-changes

Miscellaneous changes

src/error.rs

@@ -553,7 +553,7 @@ impl std::fmt::Display for ErrorKind {
             BadGuideline => write!(f, "bad guideline"),
             BadComponent => write!(f, "bad component"),
             BadImage => write!(f, "bad image"),
-            BadIdentifier => write!(f, "bad identifier"),
+            BadIdentifier => write!(f, "an identifier must be at most 100 characters long and contain only ASCII characters in the range 0x20 to 0x7E"),
             BadLib => write!(f, "bad lib"),
             UnexpectedDuplicate => write!(f, "unexpected duplicate"),
             UnexpectedMove => {

src/glyph/mod.rs

@@ -320,7 +320,8 @@ pub struct Contour {
 }
 
 impl Contour {
-    fn is_closed(&self) -> bool {
+    /// Whether the contour is closed.
+    pub fn is_closed(&self) -> bool {
         self.points.first().map_or(true, |v| v.typ != PointType::Move)
     }
 

src/identifier.rs

@@ -79,6 +79,6 @@ impl<'de> Deserialize<'de> for Identifier {
         D: Deserializer<'de>,
     {
         let string = String::deserialize(deserializer)?;
-        Identifier::new(string).map_err(|_| de::Error::custom("Identifier must be at most 100 characters long and contain only ASCII characters in the range 0x20 to 0x7E."))
+        Identifier::new(string).map_err(de::Error::custom)
     }
 }