Provide correct default for scroll_history

When implementing fallback to the default value with an u32 you will get
0 as the default value.

However the default scrollback value is 10_000. A custom deserializer
has been implemented which automatically falls back to the correct
default value.

1 files changed, 13 insertions, 1 deletions

diff --git a/src/config.rs b/src/config.rs
index 2c2cfe46..d49dd3b4 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -398,7 +398,7 @@ pub struct Config {
     tabspaces: usize,
 
     /// How much scrolling history to keep
-    #[serde(default="default_scroll_history", deserialize_with="failure_default")]
+    #[serde(default="default_scroll_history", deserialize_with="deserialize_scroll_history")]
     scroll_history: u32,
 }
 
@@ -406,6 +406,18 @@ fn default_scroll_history() -> u32 {
     10_000
 }
 
+fn deserialize_scroll_history<'a, D>(deserializer: D) -> ::std::result::Result<u32, D::Error>
+    where D: de::Deserializer<'a>
+{
+    match u32::deserialize(deserializer) {
+        Ok(lines) => Ok(lines),
+        Err(err) => {
+            eprintln!("problem with config: {}; Using default value", err);
+            Ok(default_scroll_history())
+        },
+    }
+}
+
 fn failure_default_vec<'a, D, T>(deserializer: D) -> ::std::result::Result<Vec<T>, D::Error>
     where D: de::Deserializer<'a>,
           T: Deserialize<'a>