Merge pull request #97 from demvsystems/error-handling

changed return type from option to result to display internal error

Author: SaschaScholly

src/gzip.rs

@@ -1,17 +1,17 @@
-pub fn decode(bytes: Vec<u8>) -> Option<String> {
+pub fn decode(bytes: Vec<u8>) -> Result<String, String> {
     use flate2::read::GzDecoder;
     use std::io::Read;
 
     let mut decoder = GzDecoder::new(&bytes[..]);
     let mut xml = String::new();
 
     match decoder.read_to_string(&mut xml) {
-        Ok(_) => Some(xml),
-        Err(_) => None,
+        Ok(_) => Ok(xml),
+        Err(error) => Err(error.to_string()),
     }
 }
 
-pub fn encode(content: &str) -> Option<Vec<u8>> {
+pub fn encode(content: &str) -> Result<Vec<u8>, String> {
     use flate2::read::GzEncoder;
     use flate2::Compression;
     use std::io::Read;
@@ -20,7 +20,7 @@ pub fn encode(content: &str) -> Option<Vec<u8>> {
     let mut bytes = Vec::new();
 
     match encoder.read_to_end(&mut bytes) {
-        Ok(_) => Some(bytes),
-        Err(_) => None,
+        Ok(_) => Ok(bytes),
+        Err(error) => Err(error.to_string()),
     }
 }