Auto merge of #1146 - seemyvest:fix-metadata, r=emilio

Don't unwrap header metadata

Fix for a comment in #1092.
r? @fitzgen

1 files changed, 12 insertions, 8 deletions

diff --git a/src/lib.rs b/src/lib.rs
index 3bfec0b4..35360d33 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -1607,16 +1607,20 @@ impl Bindings {
         }
 
         if let Some(h) = options.input_header.as_ref() {
-            let md = std::fs::metadata(h).ok().unwrap();
-            if !md.is_file() {
-                eprintln!("error: '{}' is a folder", h);
-                return Err(());
-            }
-            if !can_read(&md.permissions()) {
-                eprintln!("error: insufficient permissions to read '{}'", h);
+            if let Ok(md) = std::fs::metadata(h) {
+                if !md.is_file() {
+                    eprintln!("error: '{}' is a folder", h);
+                    return Err(());
+                }
+                if !can_read(&md.permissions()) {
+                    eprintln!("error: insufficient permissions to read '{}'", h);
+                    return Err(());
+                }
+                options.clang_args.push(h.clone())
+            } else {
+                eprintln!("error: header '{}' does not exist.", h);
                 return Err(());
             }
-            options.clang_args.push(h.clone())
         }
 
         for f in options.input_unsaved_files.iter() {