Merge branch 'master' into issue2894

Author: Michael Wright

CHANGELOG.md

@@ -507,7 +507,7 @@ All notable changes to this project will be documented in this file.
 ## 0.0.74 — 2016-06-07
 * Fix bug with `cargo-clippy` JSON parsing
 * Add the `CLIPPY_DISABLE_DOCS_LINKS` environment variable to deactivate the
-  “for further information visit *wiki-link*” message.
+  “for further information visit *lint-link*” message.
 
 ## 0.0.73 — 2016-06-05
 * Fix false positives in [`useless_let_if_seq`]
@@ -612,7 +612,7 @@ All notable changes to this project will be documented in this file.
 [`AsRef`]: https://doc.rust-lang.org/std/convert/trait.AsRef.html
 [configuration file]: ./rust-clippy#configuration
 
-<!-- begin autogenerated links to wiki -->
+<!-- begin autogenerated links to lint list -->
 [`absurd_extreme_comparisons`]: https://rust-lang-nursery.github.io/rust-clippy/master/index.html#absurd_extreme_comparisons
 [`almost_swapped`]: https://rust-lang-nursery.github.io/rust-clippy/master/index.html#almost_swapped
 [`approx_constant`]: https://rust-lang-nursery.github.io/rust-clippy/master/index.html#approx_constant
@@ -894,4 +894,4 @@ All notable changes to this project will be documented in this file.
 [`zero_prefixed_literal`]: https://rust-lang-nursery.github.io/rust-clippy/master/index.html#zero_prefixed_literal
 [`zero_ptr`]: https://rust-lang-nursery.github.io/rust-clippy/master/index.html#zero_ptr
 [`zero_width_space`]: https://rust-lang-nursery.github.io/rust-clippy/master/index.html#zero_width_space
-<!-- end autogenerated links to wiki -->
+<!-- end autogenerated links to lint list -->

clippy_lints/src/eta_reduction.rs

@@ -15,7 +15,11 @@ pub struct EtaPass;
 /// **Why is this bad?** Needlessly creating a closure adds code for no benefit
 /// and gives the optimizer more work.
 ///
-/// **Known problems:** None.
+/// **Known problems:** If creating the closure inside the closure has a side-
+/// effect then moving the closure creation out will change when that side-
+/// effect runs.
+/// See https://github.com/rust-lang-nursery/rust-clippy/issues/1439 for more
+/// details.
 ///
 /// **Example:**
 /// ```rust

clippy_lints/src/excessive_precision.rs

@@ -65,7 +65,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for ExcessivePrecision {
 }
 
 impl ExcessivePrecision {
-    // None if nothing to lint, Some(suggestion) if lint neccessary
+    // None if nothing to lint, Some(suggestion) if lint necessary
     fn check(&self, sym: Symbol, fty: FloatTy) -> Option<String> {
         let max = max_digits(fty);
         let sym_str = sym.as_str();

clippy_lints/src/loops.rs

@@ -2217,7 +2217,7 @@ impl<'a, 'tcx> Visitor<'tcx> for VarCollectorVisitor<'a, 'tcx> {
     fn visit_expr(&mut self, ex: &'tcx Expr) {
         match ex.node {
             ExprKind::Path(_) => self.insert_def_id(ex),
-            // If there is any fuction/method call… we just stop analysis
+            // If there is any function/method call… we just stop analysis
             ExprKind::Call(..) | ExprKind::MethodCall(..) => self.skip = true,
 
             _ => walk_expr(self, ex),

clippy_lints/src/matches.rs

@@ -383,7 +383,7 @@ fn check_wild_err_arm(cx: &LateContext<'_, '_>, ex: &Expr, arms: &[Arm]) {
                                            arm.pats[0].span,
                                            "Err(_) will match all errors, maybe not a good idea",
                                            arm.pats[0].span,
-                                           "to remove this warning, match each error seperately \
+                                           "to remove this warning, match each error separately \
                                             or use unreachable macro");
                     }
                 }

clippy_lints/src/neg_cmp_op_on_partial_ord.rs

@@ -6,21 +6,21 @@ use if_chain::if_chain;
 use crate::utils::{self, paths, span_lint};
 
 /// **What it does:**
-/// Checks for the usage of negated comparision operators on types which only implement
+/// Checks for the usage of negated comparison operators on types which only implement
 /// `PartialOrd` (e.g. `f64`).
 ///
 /// **Why is this bad?**
 /// These operators make it easy to forget that the underlying types actually allow not only three
-/// potential Orderings (Less, Equal, Greater) but also a forth one (Uncomparable). Escpeccially if
-/// the operator based comparision result is negated it is easy to miss that fact.
+/// potential Orderings (Less, Equal, Greater) but also a fourth one (Uncomparable). This is
+/// especially easy to miss if the operator based comparison result is negated.
 ///
 /// **Known problems:** None.
 ///
 /// **Example:**
 ///
 /// ```rust
-/// use core::cmp::Ordering;
-///
+/// use std::cmp::Ordering;
+/// 
 /// // Bad
 /// let a = 1.0;
 /// let b = std::f64::NAN;
@@ -39,7 +39,7 @@ use crate::utils::{self, paths, span_lint};
 declare_clippy_lint! {
     pub NEG_CMP_OP_ON_PARTIAL_ORD,
     complexity,
-    "The use of negated comparision operators on partially orded types may produce confusing code."
+    "The use of negated comparison operators on partially ordered types may produce confusing code."
 }
 
 pub struct NoNegCompOpForPartialOrd;
@@ -85,10 +85,10 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for NoNegCompOpForPartialOrd {
                         cx,
                         NEG_CMP_OP_ON_PARTIAL_ORD,
                         expr.span,
-                        "The use of negated comparision operators on partially orded \
+                        "The use of negated comparison operators on partially ordered \
                         types produces code that is hard to read and refactor. Please \
-                        consider to use the `partial_cmp` instead, to make it clear \
-                        that the two values could be incomparable."
+                        consider using the `partial_cmp` method instead, to make it \
+                        clear that the two values could be incomparable."
                     )
                 }
             }

clippy_lints/src/types.rs

@@ -700,7 +700,7 @@ declare_clippy_lint! {
 
 /// **What it does:** Checks for casts of a function pointer to a numeric type not enough to store address.
 ///
-/// **Why is this bad?** Casting a function pointer to not eligable type could truncate the address value.
+/// **Why is this bad?** Casting a function pointer to not eligible type could truncate the address value.
 ///
 /// **Known problems:** None.
 ///

clippy_lints/src/utils/sugg.rs

@@ -334,7 +334,7 @@ pub fn make_assoc(op: AssocOp, lhs: &Sugg<'_>, rhs: &Sugg<'_>) -> Sugg<'static>
     Sugg::BinOp(op, sugg.into())
 }
 
-/// Convinience wrapper arround `make_assoc` and `AssocOp::from_ast_binop`.
+/// Convenience wrapper around `make_assoc` and `AssocOp::from_ast_binop`.
 pub fn make_binop(op: ast::BinOpKind, lhs: &Sugg<'_>, rhs: &Sugg<'_>) -> Sugg<'static> {
     make_assoc(AssocOp::from_ast_binop(op), lhs, rhs)
 }

clippy_lints/src/write.rs

@@ -290,22 +290,24 @@ fn check_tts(cx: &EarlyContext<'a>, tts: &ThinTokenStream, is_write: bool) -> Op
                 idx += 1;
             },
             ExprKind::Assign(lhs, rhs) => {
-                if let ExprKind::Path(_, p) = &lhs.node {
-                    let mut all_simple = true;
-                    let mut seen = false;
-                    for arg in &args {
-                        match arg.position {
-                            | ArgumentImplicitlyIs(_)
-                            | ArgumentIs(_)
-                            => {},
-                            ArgumentNamed(name) => if *p == name {
-                                seen = true;
-                                all_simple &= arg.format == SIMPLE;
-                            },
+                if let ExprKind::Lit(_) = rhs.node {
+                    if let ExprKind::Path(_, p) = &lhs.node {
+                        let mut all_simple = true;
+                        let mut seen = false;
+                        for arg in &args {
+                            match arg.position {
+                                | ArgumentImplicitlyIs(_)
+                                | ArgumentIs(_)
+                                => {},
+                                ArgumentNamed(name) => if *p == name {
+                                    seen = true;
+                                    all_simple &= arg.format == SIMPLE;
+                                },
+                            }
+                        }
+                        if all_simple && seen {
+                            span_lint(cx, lint, rhs.span, "literal with an empty format string");
                         }
-                    }
-                    if all_simple && seen {
-                        span_lint(cx, lint, rhs.span, "literal with an empty format string");
                     }
                 }
             },

tests/ui/booleans.rs

@@ -116,7 +116,7 @@ fn warn_for_built_in_methods_with_negation() {
 }
 
 #[allow(neg_cmp_op_on_partial_ord)]
-fn dont_warn_for_negated_partial_ord_comparision() {
+fn dont_warn_for_negated_partial_ord_comparison() {
     let a: f64 = unimplemented!();
     let b: f64 = unimplemented!();
     let _ = !(a < b);