-use crate::utils::span_lint_and_sugg;
-use rustc_errors::Applicability;
-use rustc_hir::{CaptureBy, Expr, ExprKind, PatKind, QPath};
+use crate::utils::span_lint_and_help;
+
+use rustc_hir::{CaptureBy, Expr, ExprKind, PatKind};
use rustc_lint::{LateContext, LateLintPass};
use rustc_session::{declare_lint_pass, declare_tool_lint};
declare_clippy_lint! {
/// **What it does:** Checks for instances of `map_err(|_| Some::Enum)`
///
- /// **Why is this bad?** This map_err throws away the original error rather than allowing the enum to bubble the original error
+ /// **Why is this bad?** This map_err throws away the original error rather than allowing the enum to contain and report the cause of the error
///
/// **Known problems:** None.
///
/// **Example:**
- ///
+ /// Before:
/// ```rust
- /// enum Errors {
- /// Ignore
- ///}
- ///fn main() -> Result<(), Errors> {
+ /// use std::fmt;
///
- /// let x = u32::try_from(-123_i32);
+ /// #[derive(Debug)]
+ /// enum Error {
+ /// Indivisible,
+ /// Remainder(u8),
+ /// }
///
- /// println!("{:?}", x.map_err(|_| Errors::Ignore));
+ /// impl fmt::Display for Error {
+ /// fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+ /// match self {
+ /// Error::Indivisible => write!(f, "could not divide input by three"),
+ /// Error::Remainder(remainder) => write!(
+ /// f,
+ /// "input is not divisible by three, remainder = {}",
+ /// remainder
+ /// ),
+ /// }
+ /// }
+ /// }
///
- /// Ok(())
- ///}
- /// ```
- /// Use instead:
- /// ```rust
- /// enum Errors {
- /// WithContext(TryFromIntError)
- ///}
- ///fn main() -> Result<(), Errors> {
+ /// impl std::error::Error for Error {}
+ ///
+ /// fn divisible_by_3(input: &str) -> Result<(), Error> {
+ /// input
+ /// .parse::<i32>()
+ /// .map_err(|_| Error::Indivisible)
+ /// .map(|v| v % 3)
+ /// .and_then(|remainder| {
+ /// if remainder == 0 {
+ /// Ok(())
+ /// } else {
+ /// Err(Error::Remainder(remainder as u8))
+ /// }
+ /// })
+ /// }
+ /// ```
+ ///
+ /// After:
+ /// ```rust
+ /// use std::{fmt, num::ParseIntError};
+ ///
+ /// #[derive(Debug)]
+ /// enum Error {
+ /// Indivisible(ParseIntError),
+ /// Remainder(u8),
+ /// }
///
- /// let x = u32::try_from(-123_i32);
+ /// impl fmt::Display for Error {
+ /// fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+ /// match self {
+ /// Error::Indivisible(_) => write!(f, "could not divide input by three"),
+ /// Error::Remainder(remainder) => write!(
+ /// f,
+ /// "input is not divisible by three, remainder = {}",
+ /// remainder
+ /// ),
+ /// }
+ /// }
+ /// }
///
- /// println!("{:?}", x.map_err(|e| Errors::WithContext(e)));
+ /// impl std::error::Error for Error {
+ /// fn source(&self) -> Option<&(dyn std::error::Error + 'static)> {
+ /// match self {
+ /// Error::Indivisible(source) => Some(source),
+ /// _ => None,
+ /// }
+ /// }
+ /// }
///
- /// Ok(())
- ///}
+ /// fn divisible_by_3(input: &str) -> Result<(), Error> {
+ /// input
+ /// .parse::<i32>()
+ /// .map_err(Error::Indivisible)
+ /// .map(|v| v % 3)
+ /// .and_then(|remainder| {
+ /// if remainder == 0 {
+ /// Ok(())
+ /// } else {
+ /// Err(Error::Remainder(remainder as u8))
+ /// }
+ /// })
+ /// }
/// ```
pub MAP_ERR_IGNORE,
- style,
+ pedantic,
"`map_err` should not ignore the original error"
}
if closure_body.params.len() == 1 {
// make sure that parameter is the wild token (`_`)
if let PatKind::Wild = closure_body.params[0].pat.kind {
- // Check the value of the closure to see if we can build the enum we are throwing away
- // the error for make sure this is a Path
- if let ExprKind::Path(q_path) = &closure_body.value.kind {
- // this should be a resolved path, only keep the path field
- if let QPath::Resolved(_, path) = q_path {
- // finally get the idents for each path segment collect them as a string and
- // join them with the path separator ("::"")
- let closure_fold: String = path
- .segments
- .iter()
- .map(|x| x.ident.as_str().to_string())
- .collect::<Vec<String>>()
- .join("::");
- //Span the body of the closure (the |...| bit) and suggest the fix by taking
- // the error and encapsulating it in the enum
- span_lint_and_sugg(
- cx,
- MAP_ERR_IGNORE,
- body_span,
- "`map_err` has thrown away the original error",
- "Allow the error enum to encapsulate the original error",
- format!("|e| {}(e)", closure_fold),
- Applicability::HasPlaceholders,
- );
- }
- } else {
- //If we cannot build the enum in a human readable way just suggest not throwing way
- // the error
- span_lint_and_sugg(
- cx,
- MAP_ERR_IGNORE,
- body_span,
- "`map_err` has thrown away the original error",
- "Allow the error enum to encapsulate the original error",
- "|e|".to_string(),
- Applicability::HasPlaceholders,
- );
- }
+ // span the area of the closure capture and warn that the
+ // original error will be thrown away
+ span_lint_and_help(
+ cx,
+ MAP_ERR_IGNORE,
+ body_span,
+ "`map_err(|_|...` ignores the original error",
+ None,
+ "Consider wrapping the error in an enum variant",
+ );
}
}
}