]> git.lizzy.rs Git - rust.git/commitdiff
Revert "Add lint checks for unused loop labels"
authorKyle Stachowicz <kylestach99@gmail.com>
Thu, 17 May 2018 23:57:46 +0000 (16:57 -0700)
committerKyle Stachowicz <kylestach99@gmail.com>
Fri, 18 May 2018 23:57:15 +0000 (16:57 -0700)
This functionality is being reimplemented in the resolver phase

This reverts commit 503a69e844970476b27bf1ac7be951bb22194f50.

src/librustc_lint/lib.rs
src/librustc_lint/unused.rs
src/librustc_mir/interpret/eval_context.rs
src/test/ui/lint/unused_label.rs [deleted file]
src/test/ui/lint/unused_label.stderr [deleted file]

index b0a766ec058749a6f101162a11c74f1e0038531a..0ae133640fad9feb93bd68ce0faec1a271cbaa4b 100644 (file)
@@ -110,7 +110,6 @@ macro_rules! add_lint_group {
 
     add_early_builtin_with_new!(sess,
                                 DeprecatedAttr,
-                                UnusedLabel,
                                 );
 
     add_builtin!(sess,
@@ -178,7 +177,6 @@ macro_rules! add_lint_group {
                     UNUSED_DOC_COMMENT,
                     UNUSED_EXTERN_CRATES,
                     UNUSED_FEATURES,
-                    UNUSED_LABEL,
                     UNUSED_PARENS);
 
     add_lint_group!(sess,
index 2321d40f150e1e045eaddef7f48f016448af97cd..845c964b986dda3cb319b7ac12c38abe98702821 100644 (file)
@@ -464,61 +464,3 @@ fn check_expr(&mut self, cx: &LateContext, e: &hir::Expr) {
         }
     }
 }
-
-declare_lint! {
-    pub(super) UNUSED_LABEL,
-    Warn,
-    "warns on unused labels"
-}
-
-#[derive(Clone)]
-pub struct UnusedLabel(pub Vec<(ast::Label, bool)>);
-
-impl UnusedLabel {
-    pub fn new() -> Self {
-        UnusedLabel(vec![])
-    }
-}
-
-impl LintPass for UnusedLabel {
-    fn get_lints(&self) -> LintArray {
-        lint_array!(UNUSED_LABEL)
-    }
-}
-
-impl EarlyLintPass for UnusedLabel {
-    fn check_expr(&mut self, _: &EarlyContext, expr: &ast::Expr) {
-        match expr.node {
-            ast::ExprKind::While(_, _, Some(label))
-            | ast::ExprKind::WhileLet(_, _, _, Some(label))
-            | ast::ExprKind::ForLoop(_, _, _, Some(label))
-            | ast::ExprKind::Loop(_, Some(label)) => {
-                self.0.push((label, false));
-            }
-            ast::ExprKind::Break(Some(label), _) | ast::ExprKind::Continue(Some(label)) => {
-                if let Some((_, ref mut was_used)) =
-                    self.0.iter_mut().rev().find(|(l, _)| label == *l)
-                {
-                    *was_used = true;
-                }
-            }
-            _ => {}
-        }
-    }
-
-    fn check_expr_post(&mut self, ctxt: &EarlyContext, expr: &ast::Expr) {
-        match expr.node {
-            ast::ExprKind::While(_, _, Some(label))
-            | ast::ExprKind::WhileLet(_, _, _, Some(label))
-            | ast::ExprKind::ForLoop(_, _, _, Some(label))
-            | ast::ExprKind::Loop(_, Some(label)) => {
-                if let Some((_, was_used)) = self.0.pop() {
-                    if !was_used {
-                        ctxt.span_lint(UNUSED_LABEL, label.ident.span, "unused label");
-                    }
-                }
-            }
-            _ => {}
-        }
-    }
-}
index 7b6cee96a875c25e0cf37008f97434f1137312cc..03137619edaf48d72a0780b073a6446cc575ec53 100644 (file)
@@ -1705,7 +1705,7 @@ pub fn report(&self, e: &mut EvalError, as_err: bool, explicit_span: Option<Span
             let mut trace_text = "\n\nAn error occurred in miri:\n".to_string();
             backtrace.resolve();
             write!(trace_text, "backtrace frames: {}\n", backtrace.frames().len()).unwrap();
-            for (i, frame) in backtrace.frames().iter().enumerate() {
+            'frames: for (i, frame) in backtrace.frames().iter().enumerate() {
                 if frame.symbols().is_empty() {
                     write!(trace_text, "{}: no symbols\n", i).unwrap();
                 }
diff --git a/src/test/ui/lint/unused_label.rs b/src/test/ui/lint/unused_label.rs
deleted file mode 100644 (file)
index f19b793..0000000
+++ /dev/null
@@ -1,76 +0,0 @@
-// Copyright 2017 The Rust Project Developers. See the COPYRIGHT
-// file at the top-level directory of this distribution and at
-// http://rust-lang.org/COPYRIGHT.
-//
-// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
-// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
-// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
-// option. This file may not be copied, modified, or distributed
-// except according to those terms.
-
-// The output should warn when a loop label is not used. However, it
-// should also deal with the edge cases where a label is shadowed,
-// within nested loops
-
-// compile-pass
-// compile-flags: -W unused-label
-
-fn main() {
-    'unused_while_label: while 0 == 0 {
-        //~^ WARN unused label
-    }
-
-    let opt = Some(0);
-    'unused_while_let_label: while let Some(_) = opt {
-        //~^ WARN unused label
-    }
-
-    'unused_for_label: for _ in 0..10 {
-        //~^ WARN unused label
-    }
-
-    'used_loop_label: loop {
-        break 'used_loop_label;
-    }
-
-    'used_loop_label_outer_1: for _ in 0..10 {
-        'used_loop_label_inner_1: for _ in 0..10 {
-            break 'used_loop_label_inner_1;
-        }
-        break 'used_loop_label_outer_1;
-    }
-
-    'used_loop_label_outer_2: for _ in 0..10 {
-        'unused_loop_label_inner_2: for _ in 0..10 {
-            //~^ WARN unused label
-            break 'used_loop_label_outer_2;
-        }
-    }
-
-    'unused_loop_label_outer_3: for _ in 0..10 {
-        //~^ WARN unused label
-        'used_loop_label_inner_3: for _ in 0..10 {
-            break 'used_loop_label_inner_3;
-        }
-    }
-
-    // Test breaking many times with the same inner label doesn't break the
-    // warning on the outer label
-    'many_used_shadowed: for _ in 0..10 {
-        //~^ WARN unused label
-        'many_used_shadowed: for _ in 0..10 {
-            //~^ WARN label name `'many_used_shadowed` shadows a label name that is already in scope
-            if 1 % 2 == 0 {
-                break 'many_used_shadowed;
-            } else {
-                break 'many_used_shadowed;
-            }
-        }
-    }
-
-    // This is diverging, so put it at the end so we don't get
-    // unreachable_code errors everywhere else
-    'unused_loop_label: loop {
-        //~^ WARN unused label
-    }
-}
diff --git a/src/test/ui/lint/unused_label.stderr b/src/test/ui/lint/unused_label.stderr
deleted file mode 100644 (file)
index e78df67..0000000
+++ /dev/null
@@ -1,53 +0,0 @@
-warning: unused label
-  --> $DIR/unused_label.rs:19:5
-   |
-LL |     'unused_while_label: while 0 == 0 {
-   |     ^^^^^^^^^^^^^^^^^^^
-   |
-   = note: requested on the command line with `-W unused-label`
-
-warning: unused label
-  --> $DIR/unused_label.rs:24:5
-   |
-LL |     'unused_while_let_label: while let Some(_) = opt {
-   |     ^^^^^^^^^^^^^^^^^^^^^^^
-
-warning: unused label
-  --> $DIR/unused_label.rs:28:5
-   |
-LL |     'unused_for_label: for _ in 0..10 {
-   |     ^^^^^^^^^^^^^^^^^
-
-warning: unused label
-  --> $DIR/unused_label.rs:44:9
-   |
-LL |         'unused_loop_label_inner_2: for _ in 0..10 {
-   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^
-
-warning: unused label
-  --> $DIR/unused_label.rs:50:5
-   |
-LL |     'unused_loop_label_outer_3: for _ in 0..10 {
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^
-
-warning: unused label
-  --> $DIR/unused_label.rs:59:5
-   |
-LL |     'many_used_shadowed: for _ in 0..10 {
-   |     ^^^^^^^^^^^^^^^^^^^
-
-warning: unused label
-  --> $DIR/unused_label.rs:73:5
-   |
-LL |     'unused_loop_label: loop {
-   |     ^^^^^^^^^^^^^^^^^^
-
-warning: label name `'many_used_shadowed` shadows a label name that is already in scope
-  --> $DIR/unused_label.rs:61:9
-   |
-LL |     'many_used_shadowed: for _ in 0..10 {
-   |     ------------------- first declared here
-LL |         //~^ WARN unused label
-LL |         'many_used_shadowed: for _ in 0..10 {
-   |         ^^^^^^^^^^^^^^^^^^^ lifetime 'many_used_shadowed already in scope
-