From 6ed7d99453c03dc5c739018cd0c7b880cf55ea75 Mon Sep 17 00:00:00 2001
From: Harlan Lieberman-Berg <hlieberman@setec.io>
Date: Mon, 27 Feb 2017 21:49:00 -0500
Subject: [PATCH] Switch over to using font_match everywhere.

---
 font/src/ft/list_fonts.rs |  2 +-
 font/src/ft/mod.rs        | 12 ++++--------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/font/src/ft/list_fonts.rs b/font/src/ft/list_fonts.rs
index 0789bbd..fdadea8 100644
--- a/font/src/ft/list_fonts.rs
+++ b/font/src/ft/list_fonts.rs
@@ -63,7 +63,6 @@ pub mod fc {
     }
 
     /// Find the font closest matching the provided pattern.
-    #[allow(dead_code)]
     pub fn font_match(
         config: &ConfigRef,
         pattern: &mut PatternRef,
@@ -90,6 +89,7 @@ pub mod fc {
     }
 
     /// list fonts by closeness to the pattern
+    #[allow(dead_code)]
     pub fn font_sort(
         config: &ConfigRef,
         pattern: &mut PatternRef,
diff --git a/font/src/ft/mod.rs b/font/src/ft/mod.rs
index 20dfd2c..d658526 100644
--- a/font/src/ft/mod.rs
+++ b/font/src/ft/mod.rs
@@ -140,14 +140,11 @@ impl FreeTypeRasterizer {
         pattern.set_weight(weight.into_fontconfig_type());
         pattern.set_slant(slant.into_fontconfig_type());
 
-        let fonts = fc::font_sort(fc::Config::get_current(), &mut pattern)
+        let font = fc::font_match(fc::Config::get_current(), &mut pattern)
             .ok_or_else(|| Error::MissingFont(desc.to_owned()))?;
 
-        // Take first font that has a path
-        for font in &fonts {
-            if let (Some(path), Some(index)) = (font.file(0), font.index(0)) {
-                return Ok(self.library.new_face(path, index)?);
-            }
+        if let (Some(path), Some(index)) = (font.file(0), font.index(0)) {
+            return Ok(self.library.new_face(path, index)?);
         }
 
         Err(Error::MissingFont(desc.to_owned()))
@@ -167,9 +164,8 @@ impl FreeTypeRasterizer {
         if let (Some(path), Some(index)) = (font.file(0), font.index(0)) {
             println!("got font path={:?}", path);
             return Ok(self.library.new_face(path, index)?);
-        } else {
-            Err(Error::MissingFont(desc.to_owned()))
         }
+        Err(Error::MissingFont(desc.to_owned()))
     }
 
     fn get_rendered_glyph(&mut self, glyph_key: &GlyphKey, have_recursed: bool)