rust-clippy/src/open_options.rs

use rustc::lint::*;
use rustc_front::hir::{Expr, ExprMethodCall, ExprLit};
use syntax::ast::LitKind;
use syntax::codemap::{Span, Spanned};
use utils::{walk_ptrs_ty_depth, match_type, span_lint, OPEN_OPTIONS_PATH};

/// **What it does:** This lint checks for duplicate open options as well as combinations that make no sense.
///
/// **Why is this bad?** In the best case, the code will be harder to read than necessary. I don't know the worst case.
///
/// **Known problems:** None
///
/// **Example:** `OpenOptions::new().read(true).truncate(true)`
declare_lint! {
    pub NONSENSICAL_OPEN_OPTIONS,
    Warn,
    "nonsensical combination of options for opening a file"
}


#[derive(Copy,Clone)]
pub struct NonSensicalOpenOptions;

impl LintPass for NonSensicalOpenOptions {
    fn get_lints(&self) -> LintArray {
        lint_array!(NONSENSICAL_OPEN_OPTIONS)
    }
}

impl LateLintPass for NonSensicalOpenOptions {
    fn check_expr(&mut self, cx: &LateContext, e: &Expr) {
        if let ExprMethodCall(ref name, _, ref arguments) = e.node {
            let (obj_ty, _) = walk_ptrs_ty_depth(cx.tcx.expr_ty(&arguments[0]));
            if name.node.as_str() == "open" && match_type(cx, obj_ty, &OPEN_OPTIONS_PATH) {
                let mut options = Vec::new();
                get_open_options(cx, &arguments[0], &mut options);
                check_open_options(cx, &options, e.span);
            }
        }
    }
}

#[derive(Debug, PartialEq, Eq, Clone, Copy)]
enum Argument {
    True,
    False,
    Unknown,
}

#[derive(Debug)]
enum OpenOption {
    Write,
    Read,
    Truncate,
    Create,
    Append,
}

fn get_open_options(cx: &LateContext, argument: &Expr, options: &mut Vec<(OpenOption, Argument)>) {
    if let ExprMethodCall(ref name, _, ref arguments) = argument.node {
        let (obj_ty, _) = walk_ptrs_ty_depth(cx.tcx.expr_ty(&arguments[0]));

        // Only proceed if this is a call on some object of type std::fs::OpenOptions
        if match_type(cx, obj_ty, &OPEN_OPTIONS_PATH) && arguments.len() >= 2 {

            let argument_option = match arguments[1].node {
                ExprLit(ref span) => {
                    if let Spanned {node: LitKind::Bool(lit), ..} = **span {
                        if lit {
                            Argument::True
                        } else {
                            Argument::False
                        }
                    } else {
                        return; // The function is called with a literal
                                // which is not a boolean literal. This is theoretically
                                // possible, but not very likely.
                    }
                }
                _ => Argument::Unknown,
            };

            match &*name.node.as_str() {
                "create" => {
                    options.push((OpenOption::Create, argument_option));
                }
                "append" => {
                    options.push((OpenOption::Append, argument_option));
                }
                "truncate" => {
                    options.push((OpenOption::Truncate, argument_option));
                }
                "read" => {
                    options.push((OpenOption::Read, argument_option));
                }
                "write" => {
                    options.push((OpenOption::Write, argument_option));
                }
                _ => {}
            }

            get_open_options(cx, &arguments[0], options);
        }
    }
}

fn check_open_options(cx: &LateContext, options: &[(OpenOption, Argument)], span: Span) {
    let (mut create, mut append, mut truncate, mut read, mut write) = (false, false, false, false, false);
    let (mut create_arg, mut append_arg, mut truncate_arg, mut read_arg, mut write_arg) = (false,
                                                                                           false,
                                                                                           false,
                                                                                           false,
                                                                                           false);
    // This code is almost duplicated (oh, the irony), but I haven't found a way to unify it.

    for option in options {
        match *option {
            (OpenOption::Create, arg) => {
                if create {
                    span_lint(cx,
                              NONSENSICAL_OPEN_OPTIONS,
                              span,
                              "the method \"create\" is called more than once");
                } else {
                    create = true
                }
                create_arg = create_arg || (arg == Argument::True);;
            }
            (OpenOption::Append, arg) => {
                if append {
                    span_lint(cx,
                              NONSENSICAL_OPEN_OPTIONS,
                              span,
                              "the method \"append\" is called more than once");
                } else {
                    append = true
                }
                append_arg = append_arg || (arg == Argument::True);;
            }
            (OpenOption::Truncate, arg) => {
                if truncate {
                    span_lint(cx,
                              NONSENSICAL_OPEN_OPTIONS,
                              span,
                              "the method \"truncate\" is called more than once");
                } else {
                    truncate = true
                }
                truncate_arg = truncate_arg || (arg == Argument::True);
            }
            (OpenOption::Read, arg) => {
                if read {
                    span_lint(cx,
                              NONSENSICAL_OPEN_OPTIONS,
                              span,
                              "the method \"read\" is called more than once");
                } else {
                    read = true
                }
                read_arg = read_arg || (arg == Argument::True);;
            }
            (OpenOption::Write, arg) => {
                if write {
                    span_lint(cx,
                              NONSENSICAL_OPEN_OPTIONS,
                              span,
                              "the method \"write\" is called more than once");
                } else {
                    write = true
                }
                write_arg = write_arg || (arg == Argument::True);;
            }
        }
    }

    if read && truncate && read_arg && truncate_arg {
        span_lint(cx, NONSENSICAL_OPEN_OPTIONS, span, "file opened with \"truncate\" and \"read\"");
    }
    if append && truncate && append_arg && truncate_arg {
        span_lint(cx,
                  NONSENSICAL_OPEN_OPTIONS,
                  span,
                  "file opened with \"append\" and \"truncate\"");
    }
}