Merge pull request #355 from matthiasbeyer/libimagutil/zero-warnings

Libimagutil/zero warnings
This commit is contained in:
Matthias Beyer 2016-04-17 00:14:45 +02:00
commit 3a5c965152
2 changed files with 23 additions and 7 deletions

View file

@ -1,3 +1,18 @@
#![deny(
non_camel_case_types,
non_snake_case,
path_statements,
trivial_numeric_casts,
unstable_features,
unused_allocation,
unused_import_braces,
unused_imports,
unused_must_use,
unused_mut,
unused_qualifications,
while_true,
)]
#[macro_use] extern crate lazy_static; #[macro_use] extern crate lazy_static;
#[macro_use] extern crate log; #[macro_use] extern crate log;
extern crate regex; extern crate regex;

View file

@ -21,7 +21,7 @@ use std::io::stderr;
/// ``` /// ```
pub fn trace_error(e: &Error) { pub fn trace_error(e: &Error) {
print_trace_maxdepth(count_error_causes(e), e, ::std::u64::MAX); print_trace_maxdepth(count_error_causes(e), e, ::std::u64::MAX);
write!(stderr(), "\n"); write!(stderr(), "\n").ok();
} }
/// Print an Error type and its cause recursively, but only `max` levels /// Print an Error type and its cause recursively, but only `max` levels
@ -29,9 +29,10 @@ pub fn trace_error(e: &Error) {
/// Output is the same as for `trace_error()`, though there are only `max` levels printed. /// Output is the same as for `trace_error()`, though there are only `max` levels printed.
pub fn trace_error_maxdepth(e: &Error, max: u64) { pub fn trace_error_maxdepth(e: &Error, max: u64) {
let n = count_error_causes(e); let n = count_error_causes(e);
write!(stderr(), "{}/{} Levels of errors will be printed\n", (if max > n { n } else { max }), n); write!(stderr(),
"{}/{} Levels of errors will be printed\n", (if max > n { n } else { max }), n).ok();
print_trace_maxdepth(n, e, max); print_trace_maxdepth(n, e, max);
write!(stderr(), ""); write!(stderr(), "").ok();
} }
/// Print an Error type and its cause recursively with the debug!() macro /// Print an Error type and its cause recursively with the debug!() macro
@ -48,13 +49,13 @@ pub fn trace_error_dbg(e: &Error) {
fn print_trace_maxdepth(idx: u64, e: &Error, max: u64) -> Option<&Error> { fn print_trace_maxdepth(idx: u64, e: &Error, max: u64) -> Option<&Error> {
if e.cause().is_some() && idx > 0 { if e.cause().is_some() && idx > 0 {
match print_trace_maxdepth(idx - 1, e.cause().unwrap(), max) { match print_trace_maxdepth(idx - 1, e.cause().unwrap(), max) {
None => write!(stderr(), "\n"), None => write!(stderr(), "\n").ok(),
Some(_) => write!(stderr(), " -- caused:\n"), Some(_) => write!(stderr(), " -- caused:\n").ok(),
}; };
} else { } else {
write!(stderr(), "\n"); write!(stderr(), "\n").ok();
} }
write!(stderr(), "Error {:>4} : {}", idx, e.description()); write!(stderr(), "Error {:>4} : {}", idx, e.description()).ok();
e.cause() e.cause()
} }