diff options
author | Christian Duerr <chrisduerr@users.noreply.github.com> | 2018-03-04 22:40:15 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-03-04 22:40:15 +0000 |
commit | 7f2b398ad2084bdaaa266e8da770a213f0a9a2eb (patch) | |
tree | 9d249fd13e80c9ca2d03f37fd6147fbc1ee8ba4a /src/logging.rs | |
parent | 475ebecfc4e0648242e82e256dee1489f3a3fe81 (diff) | |
download | alacritty-7f2b398ad2084bdaaa266e8da770a213f0a9a2eb.tar.gz alacritty-7f2b398ad2084bdaaa266e8da770a213f0a9a2eb.zip |
Remove all instances of unwrap() from config
Unwrapping inside the config file parsing can lead to some issues that
prevent us from falling back to a default configuration file.
One instance of that issue was mentioned in #1135.
Now all instances of `unwrap()` have been removed and replaced with
proper error handling. This will make the config more robust and
prevents live reload from silently breaking while alacritty is running.
This also fixes a few currently existing clippy issues.
Clippy added an additonal lint which complains about `MyStruct { field:
field }`.
These issues have been fixed, except for some false-positives and issues
in external macros which will probably be fixed with future updates (rust-lang-nursery/bitflags#149)
Diffstat (limited to 'src/logging.rs')
-rw-r--r-- | src/logging.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/logging.rs b/src/logging.rs index ff2a7735..421223e8 100644 --- a/src/logging.rs +++ b/src/logging.rs @@ -32,7 +32,7 @@ impl<T: Send + io::Write> Logger<T> { #[cfg_attr(feature = "clippy", allow(new_ret_no_self))] pub fn new(output: T, level: log::LevelFilter) -> Logger<io::LineWriter<T>> { Logger { - level: level, + level, output: sync::Mutex::new(io::LineWriter::new(output)) } } |