[INFO] cloning repository https://github.com/Pureblackkk/craft-interpreter-practice-rs [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/Pureblackkk/craft-interpreter-practice-rs" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FPureblackkk%2Fcraft-interpreter-practice-rs", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FPureblackkk%2Fcraft-interpreter-practice-rs'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 4659f7efe819e6cb3009b97cd7c64d14da643096 [INFO] linting Pureblackkk/craft-interpreter-practice-rs against nightly for clippy-nonminimal_bool-denied [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FPureblackkk%2Fcraft-interpreter-practice-rs" "/workspace/builds/worker-7-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/Pureblackkk/craft-interpreter-practice-rs [INFO] finished tweaking git repo https://github.com/Pureblackkk/craft-interpreter-practice-rs [INFO] tweaked toml for git repo https://github.com/Pureblackkk/craft-interpreter-practice-rs written to /workspace/builds/worker-7-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/Pureblackkk/craft-interpreter-practice-rs on toolchain nightly [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/Pureblackkk/craft-interpreter-practice-rs already has a lockfile, it will not be regenerated [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+nightly" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 787147022edcc0ef4e453d6f774e19d1cab0ab84c9566a26b128f720c2db8079 [INFO] running `Command { std: "docker" "start" "-a" "787147022edcc0ef4e453d6f774e19d1cab0ab84c9566a26b128f720c2db8079", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "787147022edcc0ef4e453d6f774e19d1cab0ab84c9566a26b128f720c2db8079", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "787147022edcc0ef4e453d6f774e19d1cab0ab84c9566a26b128f720c2db8079", kill_on_drop: false }` [INFO] [stdout] 787147022edcc0ef4e453d6f774e19d1cab0ab84c9566a26b128f720c2db8079 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "clippy" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] d95fc17780d21420c1828570598334e49d3d4d5b201e31fc1702346f6924794e [INFO] running `Command { std: "docker" "start" "-a" "d95fc17780d21420c1828570598334e49d3d4d5b201e31fc1702346f6924794e", kill_on_drop: false }` [INFO] [stderr] Compiling proc-macro2 v1.0.82 [INFO] [stderr] Compiling siphasher v0.3.11 [INFO] [stderr] Compiling rand_core v0.6.4 [INFO] [stderr] Checking phf_shared v0.11.2 [INFO] [stderr] Compiling rand v0.8.5 [INFO] [stderr] Compiling phf_generator v0.11.2 [INFO] [stderr] Compiling quote v1.0.36 [INFO] [stderr] Compiling syn v2.0.61 [INFO] [stderr] Compiling phf_macros v0.11.2 [INFO] [stderr] Checking phf v0.11.2 [INFO] [stderr] Checking craft_interpreter v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `std::fmt::Arguments` [INFO] [stdout] --> src/vistor/ast_printer.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use std::fmt::Arguments; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::rc::Rc` [INFO] [stdout] --> src/vistor/expr_interpreter.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::rc::Rc; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::value::class::ClassInstance` [INFO] [stdout] --> src/vistor/expr_interpreter.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use crate::value::class::ClassInstance; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: redundant field names in struct initialization [INFO] [stdout] --> src/vistor/expr_interpreter.rs:27:21 [INFO] [stdout] | [INFO] [stdout] 27 | message: message, [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: replace it with: `message` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stdout] = note: `#[warn(clippy::redundant_field_names)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SetValueType` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:6:62 [INFO] [stdout] | [INFO] [stdout] 6 | use crate::resolver::{ClassStatus, FunctionStatus, Resolver, SetValueType}; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: denote infinite loops with `loop { ... }` [INFO] [stdout] --> src/parser/parser_expr.rs:168:9 [INFO] [stdout] | [INFO] [stdout] 168 | while true { [INFO] [stdout] | ^^^^^^^^^^ help: use `loop` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(while_true)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: redundant field names in struct initialization [INFO] [stdout] --> src/parser/mod.rs:17:13 [INFO] [stdout] | [INFO] [stdout] 17 | tokens: tokens, [INFO] [stdout] | ^^^^^^^^^^^^^^ help: replace it with: `tokens` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fmt::Arguments` [INFO] [stdout] --> src/vistor/ast_printer.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use std::fmt::Arguments; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `cell::RefCell` [INFO] [stdout] --> src/value/mod.rs:6:11 [INFO] [stdout] | [INFO] [stdout] 6 | use std::{cell::RefCell, rc::Rc}; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `RefMut` [INFO] [stdout] --> src/environment/mod.rs:1:26 [INFO] [stdout] | [INFO] [stdout] 1 | use std::cell::{RefCell, RefMut}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::rc::Rc` [INFO] [stdout] --> src/vistor/expr_interpreter.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use std::rc::Rc; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::ops::Deref` [INFO] [stdout] --> src/environment/mod.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::ops::Deref; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::ptr` [INFO] [stdout] --> src/environment/mod.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use std::ptr; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::value::class::ClassInstance` [INFO] [stdout] --> src/vistor/expr_interpreter.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use crate::value::class::ClassInstance; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: redundant field names in struct initialization [INFO] [stdout] --> src/vistor/expr_interpreter.rs:27:21 [INFO] [stdout] | [INFO] [stdout] 27 | message: message, [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: replace it with: `message` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stdout] = note: `#[warn(clippy::redundant_field_names)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `SetValueType` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:6:62 [INFO] [stdout] | [INFO] [stdout] 6 | use crate::resolver::{ClassStatus, FunctionStatus, Resolver, SetValueType}; [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: denote infinite loops with `loop { ... }` [INFO] [stdout] --> src/parser/parser_expr.rs:168:9 [INFO] [stdout] | [INFO] [stdout] 168 | while true { [INFO] [stdout] | ^^^^^^^^^^ help: use `loop` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(while_true)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: redundant field names in struct initialization [INFO] [stdout] --> src/parser/mod.rs:17:13 [INFO] [stdout] | [INFO] [stdout] 17 | tokens: tokens, [INFO] [stdout] | ^^^^^^^^^^^^^^ help: replace it with: `tokens` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `cell::RefCell` [INFO] [stdout] --> src/value/mod.rs:6:11 [INFO] [stdout] | [INFO] [stdout] 6 | use std::{cell::RefCell, rc::Rc}; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `RefMut` [INFO] [stdout] --> src/environment/mod.rs:1:26 [INFO] [stdout] | [INFO] [stdout] 1 | use std::cell::{RefCell, RefMut}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::ops::Deref` [INFO] [stdout] --> src/environment/mod.rs:3:5 [INFO] [stdout] | [INFO] [stdout] 3 | use std::ops::Deref; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::ptr` [INFO] [stdout] --> src/environment/mod.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use std::ptr; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `token` [INFO] [stdout] --> src/vistor/ast_printer.rs:79:32 [INFO] [stdout] | [INFO] [stdout] 79 | Expr::Call(callee, token, arguments) => { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_token` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/ast_printer.rs:91:31 [INFO] [stdout] | [INFO] [stdout] 91 | Expr::Get(object, property) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/ast_printer.rs:97:31 [INFO] [stdout] | [INFO] [stdout] 97 | Expr::Set(object, property, value) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `token` [INFO] [stdout] --> src/vistor/expr_interpreter.rs:190:23 [INFO] [stdout] | [INFO] [stdout] 190 | Expr::New(token, caller) => { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_token` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `value_result` [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:63:25 [INFO] [stdout] | [INFO] [stdout] 63 | let value_result = self.exectue(statement)?; [INFO] [stdout] | ^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_value_result` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `supper_class_expr` [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:135:29 [INFO] [stdout] | [INFO] [stdout] 135 | if let Some(supper_class_expr) = supper_class { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_supper_class_expr` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `supper_class_expr` [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:171:29 [INFO] [stdout] | [INFO] [stdout] 171 | if let Some(supper_class_expr) = supper_class { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_supper_class_expr` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/expr_resolver.rs:77:31 [INFO] [stdout] | [INFO] [stdout] 77 | Expr::Get(object, property) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/expr_resolver.rs:81:31 [INFO] [stdout] | [INFO] [stdout] 81 | Expr::Set(object, property, val) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `method` [INFO] [stdout] --> src/vistor/expr_resolver.rs:97:32 [INFO] [stdout] | [INFO] [stdout] 97 | Expr::Super(token, method) => { [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_method` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable pattern [INFO] [stdout] --> src/vistor/stmt_resolver.rs:173:13 [INFO] [stdout] | [INFO] [stdout] 173 | _ => Ok(()), [INFO] [stdout] | ^ no value can reach this [INFO] [stdout] | [INFO] [stdout] note: multiple earlier patterns match some of the same values [INFO] [stdout] --> src/vistor/stmt_resolver.rs:173:13 [INFO] [stdout] | [INFO] [stdout] 58 | Stmt::Block(statements) => { [INFO] [stdout] | ----------------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 64 | Stmt::Var(indentifier, expr) => { [INFO] [stdout] | ---------------------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 75 | Stmt::Function(name, _, _) => { [INFO] [stdout] | -------------------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 82 | Stmt::Expr(expr) => { [INFO] [stdout] | ---------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 173 | _ => Ok(()), [INFO] [stdout] | ^ ...and 5 other patterns collectively make this unreachable [INFO] [stdout] = note: `#[warn(unreachable_patterns)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `supper_class_expr` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:164:29 [INFO] [stdout] | [INFO] [stdout] 164 | if let Some(supper_class_expr) = supper_class { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_supper_class_expr` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `stmt` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:177:29 [INFO] [stdout] | [INFO] [stdout] 177 | fn visit_env(&mut self, stmt: &Stmt, env: Rc>) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_stmt` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `env` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:177:42 [INFO] [stdout] | [INFO] [stdout] 177 | fn visit_env(&mut self, stmt: &Stmt, env: Rc>) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_env` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `function_type` [INFO] [stdout] --> src/parser/parser_stmt.rs:60:28 [INFO] [stdout] | [INFO] [stdout] 60 | fn function(&mut self, function_type: FunctionType) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_function_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/parser/parser_stmt.rs:170:13 [INFO] [stdout] | [INFO] [stdout] 170 | let mut initializer: Option; [INFO] [stdout] | ----^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `token` [INFO] [stdout] --> src/value/function.rs:66:9 [INFO] [stdout] | [INFO] [stdout] 66 | token: &Token, [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_token` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `res` [INFO] [stdout] --> src/interpreter/mod.rs:29:17 [INFO] [stdout] | [INFO] [stdout] 29 | let res = self.exectue(&statement)?; [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_res` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `ScannerError` is never constructed [INFO] [stdout] --> src/scanner/mod.rs:11:12 [INFO] [stdout] | [INFO] [stdout] 11 | pub struct ScannerError { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Scanner` is never constructed [INFO] [stdout] --> src/scanner/mod.rs:16:12 [INFO] [stdout] | [INFO] [stdout] 16 | pub struct Scanner { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: multiple associated items are never used [INFO] [stdout] --> src/scanner/mod.rs:27:12 [INFO] [stdout] | [INFO] [stdout] 26 | impl Scanner { [INFO] [stdout] | ------------ associated items in this implementation [INFO] [stdout] 27 | pub fn new(src: String) -> Scanner { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 39 | pub fn scan_tokens(&mut self) -> Result, ScannerError> { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 60 | fn scan_token(&mut self) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 137 | fn advance(&mut self) -> char { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 144 | fn match_next(&mut self, expected: char) -> bool { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 158 | fn peek(&mut self) -> char { [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 166 | fn peek_next(&mut self) -> char { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 174 | fn add_token(&mut self, token_type: TokenType, literal: Option) { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 187 | fn string(&mut self) { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 210 | fn number(&mut self) { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 231 | fn identifier(&mut self) { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 248 | fn is_at_end(&self) -> bool { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 252 | fn done(&self) -> bool { [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 256 | fn is_digit(c: char) -> bool { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 260 | fn is_alpha(c: char) -> bool { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 266 | fn is_alpha_numberic(c: char) -> bool { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `TokenType` is never used [INFO] [stdout] --> src/scanner/token.rs:5:10 [INFO] [stdout] | [INFO] [stdout] 5 | pub enum TokenType { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `Literal` is never used [INFO] [stdout] --> src/scanner/token.rs:61:10 [INFO] [stdout] | [INFO] [stdout] 61 | pub enum Literal { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: static `RESERVED_KEYWORD` is never used [INFO] [stdout] --> src/scanner/token.rs:67:12 [INFO] [stdout] | [INFO] [stdout] 67 | pub static RESERVED_KEYWORD: phf::Map<&'static str, TokenType> = phf_map!{ [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Token` is never constructed [INFO] [stdout] --> src/scanner/token.rs:90:12 [INFO] [stdout] | [INFO] [stdout] 90 | pub struct Token { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `Expr` is never used [INFO] [stdout] --> src/grammer/expression.rs:4:10 [INFO] [stdout] | [INFO] [stdout] 4 | pub enum Expr { [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `ExprVistor` is never used [INFO] [stdout] --> src/grammer/expression.rs:20:11 [INFO] [stdout] | [INFO] [stdout] 20 | pub trait ExprVistor { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `ExprAccept` is never used [INFO] [stdout] --> src/grammer/expression.rs:24:11 [INFO] [stdout] | [INFO] [stdout] 24 | pub trait ExprAccept { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `ExprLiteral` is never used [INFO] [stdout] --> src/grammer/expression.rs:35:10 [INFO] [stdout] | [INFO] [stdout] 35 | pub enum ExprLiteral { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `Stmt` is never used [INFO] [stdout] --> src/grammer/statement.rs:9:10 [INFO] [stdout] | [INFO] [stdout] 9 | pub enum Stmt { [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `FunctionType` is never used [INFO] [stdout] --> src/grammer/statement.rs:22:10 [INFO] [stdout] | [INFO] [stdout] 22 | pub enum FunctionType { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `StmtVistor` is never used [INFO] [stdout] --> src/grammer/statement.rs:27:11 [INFO] [stdout] | [INFO] [stdout] 27 | pub trait StmtVistor { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `StmtAccept` is never used [INFO] [stdout] --> src/grammer/statement.rs:33:11 [INFO] [stdout] | [INFO] [stdout] 33 | pub trait StmtAccept { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `AstPrinter` is never constructed [INFO] [stdout] --> src/vistor/ast_printer.rs:5:12 [INFO] [stdout] | [INFO] [stdout] 5 | pub struct AstPrinter; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `print`, and `parenthesize` are never used [INFO] [stdout] --> src/vistor/ast_printer.rs:8:12 [INFO] [stdout] | [INFO] [stdout] 7 | impl AstPrinter { [INFO] [stdout] | --------------- associated items in this implementation [INFO] [stdout] 8 | pub fn new() -> AstPrinter { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 12 | pub fn print(&mut self, expr: Expr) -> String { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 18 | fn parenthesize(&mut self, name: &str, exprs: &Vec<&Expr>) -> String { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `interpret_expr_debug`, `evaluate`, `result_expr_helper`, and `lookup_variable` are never used [INFO] [stdout] --> src/vistor/expr_interpreter.rs:14:12 [INFO] [stdout] | [INFO] [stdout] 13 | impl Interpreter { [INFO] [stdout] | ---------------- associated items in this implementation [INFO] [stdout] 14 | pub fn interpret_expr_debug(&mut self, expr: &Expr) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 18 | pub fn evaluate(&mut self, expr: &Expr) -> Result{ [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 22 | fn result_expr_helper(operation_result: Result, token: &Token) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 34 | fn lookup_variable(&mut self, name: &Token) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `interpret_stmt_debug`, `exectue`, and `exectue_with_env` are never used [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:15:12 [INFO] [stdout] | [INFO] [stdout] 14 | impl Interpreter { [INFO] [stdout] | ---------------- methods in this implementation [INFO] [stdout] 15 | pub fn interpret_stmt_debug(&mut self, stmt: &Stmt) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 19 | pub fn exectue(&mut self, stmt: &Stmt) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 23 | pub fn exectue_with_env(&mut self, stmt: &Stmt, env: Rc>) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `resolve_expr` and `resolve_local` are never used [INFO] [stdout] --> src/vistor/expr_resolver.rs:7:12 [INFO] [stdout] | [INFO] [stdout] 6 | impl Resolver<'_> { [INFO] [stdout] | ----------------- methods in this implementation [INFO] [stdout] 7 | pub fn resolve_expr(&mut self, expr: &Expr) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 11 | fn resolve_local(&mut self, name: &Token) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `resolve_stmt`, `resolve_stmt_list`, and `resolve_stmt_function` are never used [INFO] [stdout] --> src/vistor/stmt_resolver.rs:11:12 [INFO] [stdout] | [INFO] [stdout] 10 | impl Resolver<'_> { [INFO] [stdout] | ----------------- methods in this implementation [INFO] [stdout] 11 | pub fn resolve_stmt(&mut self, stmt: &Stmt) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 15 | pub fn resolve_stmt_list(&mut self, stmts: &Vec) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 23 | pub fn resolve_stmt_function(&mut self, stmt: &Stmt, function_status: FunctionStatus) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Parser` is never constructed [INFO] [stdout] --> src/parser/mod.rs:9:12 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct Parser { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: multiple associated items are never used [INFO] [stdout] --> src/parser/mod.rs:15:12 [INFO] [stdout] | [INFO] [stdout] 14 | impl Parser { [INFO] [stdout] | ----------- associated items in this implementation [INFO] [stdout] 15 | pub fn new(tokens: Vec) -> Parser { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 22 | pub fn parser(&mut self) -> Result, ParserError> { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 33 | fn consume(&mut self, typee: TokenType, message: &str) -> Result<&Token, ParserError> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 45 | fn match_one_of(&mut self, types: Vec) -> bool { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 55 | fn matches(&mut self, typee: TokenType) -> bool { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 65 | fn check(&mut self, typee: TokenType) -> bool { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 73 | fn advance(&mut self) -> &Token { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 82 | fn is_at_end(&mut self) -> bool { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 86 | fn peek(&mut self) -> &Token { [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 90 | fn previous(&mut self) -> &Token { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: multiple methods are never used [INFO] [stdout] --> src/parser/parser_expr.rs:6:12 [INFO] [stdout] | [INFO] [stdout] 5 | impl Parser { [INFO] [stdout] | ----------- methods in this implementation [INFO] [stdout] 6 | pub fn parse_expr_debug(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 10 | pub fn expression(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 14 | fn assignment(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 37 | fn or(&mut self) -> Result { [INFO] [stdout] | ^^ [INFO] [stdout] ... [INFO] [stdout] 49 | fn and(&mut self) -> Result { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 61 | fn equality(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 80 | fn comparision(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 102 | fn term(&mut self) -> Result { [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 121 | fn factor(&mut self) -> Result { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 140 | fn unary(&mut self) -> Result { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 154 | fn class_init(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 165 | fn call(&mut self) -> Result { [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 186 | fn primary(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `finsh_call` is never used [INFO] [stdout] --> src/parser/parser_expr.rs:259:8 [INFO] [stdout] | [INFO] [stdout] 258 | impl Parser { [INFO] [stdout] | ----------- method in this implementation [INFO] [stdout] 259 | fn finsh_call(&mut self, callee: Expr) -> Result { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: multiple methods are never used [INFO] [stdout] --> src/parser/parser_stmt.rs:6:12 [INFO] [stdout] | [INFO] [stdout] 5 | impl Parser { [INFO] [stdout] | ----------- methods in this implementation [INFO] [stdout] 6 | pub fn declaration(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 23 | fn var_declaration(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 36 | fn class(&mut self) -> Result { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 60 | fn function(&mut self, function_type: FunctionType) -> Result { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 89 | fn statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 117 | fn block(&mut self) -> Result { [INFO] [stdout] | ^^^^^ [INFO] [stdout] ... [INFO] [stdout] 130 | fn print_statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 136 | fn expression_statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 142 | fn if_statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 157 | fn while_statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 166 | fn for_statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 229 | fn return_statement(&mut self) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `ParserError` is never used [INFO] [stdout] --> src/parser/parese_error.rs:5:10 [INFO] [stdout] | [INFO] [stdout] 5 | pub enum ParserError { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `LValue` is never used [INFO] [stdout] --> src/value/mod.rs:12:10 [INFO] [stdout] | [INFO] [stdout] 12 | pub enum LValue { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `IsTruthy` is never used [INFO] [stdout] --> src/value/condition.rs:3:11 [INFO] [stdout] | [INFO] [stdout] 3 | pub trait IsTruthy { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Function` is never constructed [INFO] [stdout] --> src/value/function.rs:13:12 [INFO] [stdout] | [INFO] [stdout] 13 | pub struct Function { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `bind` is never used [INFO] [stdout] --> src/value/function.rs:35:12 [INFO] [stdout] | [INFO] [stdout] 34 | impl Function { [INFO] [stdout] | ------------- method in this implementation [INFO] [stdout] 35 | pub fn bind(self: &Rc, instance: Rc) -> Rc { [INFO] [stdout] | ^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: trait `Callable` is never used [INFO] [stdout] --> src/value/function.rs:52:11 [INFO] [stdout] | [INFO] [stdout] 52 | pub trait Callable { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Class` is never constructed [INFO] [stdout] --> src/value/class.rs:14:12 [INFO] [stdout] | [INFO] [stdout] 14 | pub struct Class { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `to_string`, and `find_method` are never used [INFO] [stdout] --> src/value/class.rs:28:12 [INFO] [stdout] | [INFO] [stdout] 27 | impl Class { [INFO] [stdout] | ---------- associated items in this implementation [INFO] [stdout] 28 | pub fn new( [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 40 | pub fn to_string(&self) -> String { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 44 | pub fn find_method(self: Rc, token: &Token) -> Option> { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `call` and `arity` are never used [INFO] [stdout] --> src/value/class.rs:62:12 [INFO] [stdout] | [INFO] [stdout] 61 | impl Class { [INFO] [stdout] | ---------- methods in this implementation [INFO] [stdout] 62 | pub fn call( [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 77 | pub fn arity(&self) -> usize { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `ClassInstance` is never constructed [INFO] [stdout] --> src/value/class.rs:83:12 [INFO] [stdout] | [INFO] [stdout] 83 | pub struct ClassInstance { [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `to_string`, `get`, and `set` are never used [INFO] [stdout] --> src/value/class.rs:96:12 [INFO] [stdout] | [INFO] [stdout] 95 | impl ClassInstance { [INFO] [stdout] | ------------------ associated items in this implementation [INFO] [stdout] 96 | pub fn new(class: Rc, [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 124 | pub fn to_string(&self) -> String { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 128 | pub fn get(self: Rc, name: &Token) -> Result { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 146 | pub fn set(self: Rc, name: &Token, value: LValue) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `RunTimeError` is never used [INFO] [stdout] --> src/runner/error.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | pub enum RunTimeError { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `CommonError` is never constructed [INFO] [stdout] --> src/runner/error.rs:13:12 [INFO] [stdout] | [INFO] [stdout] 13 | pub struct CommonError { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Interpreter` is never constructed [INFO] [stdout] --> src/interpreter/mod.rs:9:12 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct Interpreter { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `interpret`, and `resolve` are never used [INFO] [stdout] --> src/interpreter/mod.rs:16:12 [INFO] [stdout] | [INFO] [stdout] 15 | impl Interpreter { [INFO] [stdout] | ---------------- associated items in this implementation [INFO] [stdout] 16 | pub fn new() -> Interpreter { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 27 | pub fn interpret(&mut self, statements: Vec) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 35 | pub fn resolve(&mut self, token: &Token, depth: usize) { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Environment` is never constructed [INFO] [stdout] --> src/environment/mod.rs:11:12 [INFO] [stdout] | [INFO] [stdout] 11 | pub struct Environment { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `new`, `define`, `assign`, `get`, `get_at`, and `assign_at` are never used [INFO] [stdout] --> src/environment/mod.rs:17:12 [INFO] [stdout] | [INFO] [stdout] 16 | impl Environment { [INFO] [stdout] | ---------------- associated items in this implementation [INFO] [stdout] 17 | pub fn new() -> Environment { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 24 | pub fn define(&mut self, name: String, value: LValue) { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 28 | pub fn assign(&mut self, token: &Token, value: LValue) -> Result<(), RunTimeError>{ [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 50 | pub fn get(&self, token: &Token) -> Result{ [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 71 | pub fn get_at(&self, distance: usize, token: &Token) -> Result { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 89 | pub fn assign_at(&mut self, distance: usize, token: &Token, value: LValue) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `Resolver` is never constructed [INFO] [stdout] --> src/resolver/mod.rs:10:12 [INFO] [stdout] | [INFO] [stdout] 10 | pub struct Resolver<'a> { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `SetValueType` is never used [INFO] [stdout] --> src/resolver/mod.rs:18:10 [INFO] [stdout] | [INFO] [stdout] 18 | pub enum SetValueType { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `FunctionStatus` is never used [INFO] [stdout] --> src/resolver/mod.rs:24:10 [INFO] [stdout] | [INFO] [stdout] 24 | pub enum FunctionStatus { [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `ClassStatus` is never used [INFO] [stdout] --> src/resolver/mod.rs:32:10 [INFO] [stdout] | [INFO] [stdout] 32 | pub enum ClassStatus { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: multiple associated items are never used [INFO] [stdout] --> src/resolver/mod.rs:38:12 [INFO] [stdout] | [INFO] [stdout] 37 | impl<'a> Resolver<'a> { [INFO] [stdout] | --------------------- associated items in this implementation [INFO] [stdout] 38 | pub fn new(interpreter: &'a mut Interpreter) -> Resolver { [INFO] [stdout] | ^^^ [INFO] [stdout] ... [INFO] [stdout] 50 | pub fn resolve(&mut self, statements: &Vec) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 54 | pub fn begin_scope(&mut self) { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 58 | pub fn end_scope(&mut self) { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 62 | pub fn is_scope_empty(&mut self) -> bool { [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 66 | pub fn scope_size(&self) -> usize { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 70 | pub fn set_current_val(&mut self, token: &Token, val: bool, operation: SetValueType) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 98 | pub fn get_current_val(&mut self, name: String) -> Option { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 108 | pub fn declare(&mut self, name: &Token) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 116 | pub fn define(&mut self, name: &Token) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enum `ResolveError` is never used [INFO] [stdout] --> src/resolver/resolve_error.rs:4:10 [INFO] [stdout] | [INFO] [stdout] 4 | pub enum ResolveError { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: using `write!()` with a format string that ends in a single newline [INFO] [stdout] --> src/scanner/token.rs:100:9 [INFO] [stdout] | [INFO] [stdout] 100 | / write!( [INFO] [stdout] 101 | | f, [INFO] [stdout] 102 | | "Token {{ type: {:?}, lexeme: \"{}\", literal: {:#?}, line: {:#?}, col: {:#?}}}\n", [INFO] [stdout] 103 | | self.typee, [INFO] [stdout] ... | [INFO] [stdout] 107 | | self.col, [INFO] [stdout] 108 | | ) [INFO] [stdout] | |_________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#write_with_newline [INFO] [stdout] = note: `#[warn(clippy::write_with_newline)]` on by default [INFO] [stdout] help: use `writeln!` instead [INFO] [stdout] | [INFO] [stdout] 100 ~ writeln!( [INFO] [stdout] 101 | f, [INFO] [stdout] 102 ~ "Token {{ type: {:?}, lexeme: \"{}\", literal: {:#?}, line: {:#?}, col: {:#?}}}", [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `token` [INFO] [stdout] --> src/vistor/ast_printer.rs:79:32 [INFO] [stdout] | [INFO] [stdout] 79 | Expr::Call(callee, token, arguments) => { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_token` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/ast_printer.rs:91:31 [INFO] [stdout] | [INFO] [stdout] 91 | Expr::Get(object, property) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/ast_printer.rs:97:31 [INFO] [stdout] | [INFO] [stdout] 97 | Expr::Set(object, property, value) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual `RangeInclusive::contains` implementation [INFO] [stdout] --> src/scanner/mod.rs:257:9 [INFO] [stdout] | [INFO] [stdout] 257 | c >= '0' && c <= '9' [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: use: `('0'..='9').contains(&c)` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains [INFO] [stdout] = note: `#[warn(clippy::manual_range_contains)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual `RangeInclusive::contains` implementation [INFO] [stdout] --> src/scanner/mod.rs:261:9 [INFO] [stdout] | [INFO] [stdout] 261 | (c >= 'a' && c <= 'z') || [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ help: use: `('a'..='z').contains(&c)` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual `RangeInclusive::contains` implementation [INFO] [stdout] --> src/scanner/mod.rs:262:9 [INFO] [stdout] | [INFO] [stdout] 262 | (c >= 'A' && c <= 'Z') || [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ help: use: `('A'..='Z').contains(&c)` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/scanner/mod.rs:267:9 [INFO] [stdout] | [INFO] [stdout] 267 | return Scanner::is_alpha(c) || Scanner::is_digit(c) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] = note: `#[warn(clippy::needless_return)]` on by default [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 267 - return Scanner::is_alpha(c) || Scanner::is_digit(c) [INFO] [stdout] 267 + Scanner::is_alpha(c) || Scanner::is_digit(c) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `token` [INFO] [stdout] --> src/vistor/expr_interpreter.rs:190:23 [INFO] [stdout] | [INFO] [stdout] 190 | Expr::New(token, caller) => { [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_token` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `value_result` [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:63:25 [INFO] [stdout] | [INFO] [stdout] 63 | let value_result = self.exectue(statement)?; [INFO] [stdout] | ^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_value_result` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `supper_class_expr` [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:135:29 [INFO] [stdout] | [INFO] [stdout] 135 | if let Some(supper_class_expr) = supper_class { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_supper_class_expr` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `supper_class_expr` [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:171:29 [INFO] [stdout] | [INFO] [stdout] 171 | if let Some(supper_class_expr) = supper_class { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_supper_class_expr` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calling `push_str()` using a single-character string literal [INFO] [stdout] --> src/vistor/ast_printer.rs:20:9 [INFO] [stdout] | [INFO] [stdout] 20 | output.push_str("("); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `output.push('(')` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str [INFO] [stdout] = note: `#[warn(clippy::single_char_add_str)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/expr_resolver.rs:77:31 [INFO] [stdout] | [INFO] [stdout] 77 | Expr::Get(object, property) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `property` [INFO] [stdout] --> src/vistor/expr_resolver.rs:81:31 [INFO] [stdout] | [INFO] [stdout] 81 | Expr::Set(object, property, val) => { [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_property` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calling `push_str()` using a single-character string literal [INFO] [stdout] --> src/vistor/ast_printer.rs:24:13 [INFO] [stdout] | [INFO] [stdout] 24 | output.push_str(" "); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `output.push(' ')` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calling `push_str()` using a single-character string literal [INFO] [stdout] --> src/vistor/ast_printer.rs:28:9 [INFO] [stdout] | [INFO] [stdout] 28 | output.push_str(")"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `output.push(')')` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `method` [INFO] [stdout] --> src/vistor/expr_resolver.rs:97:32 [INFO] [stdout] | [INFO] [stdout] 97 | Expr::Super(token, method) => { [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_method` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable pattern [INFO] [stdout] --> src/vistor/stmt_resolver.rs:173:13 [INFO] [stdout] | [INFO] [stdout] 173 | _ => Ok(()), [INFO] [stdout] | ^ no value can reach this [INFO] [stdout] | [INFO] [stdout] note: multiple earlier patterns match some of the same values [INFO] [stdout] --> src/vistor/stmt_resolver.rs:173:13 [INFO] [stdout] | [INFO] [stdout] 58 | Stmt::Block(statements) => { [INFO] [stdout] | ----------------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 64 | Stmt::Var(indentifier, expr) => { [INFO] [stdout] | ---------------------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 75 | Stmt::Function(name, _, _) => { [INFO] [stdout] | -------------------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 82 | Stmt::Expr(expr) => { [INFO] [stdout] | ---------------- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 173 | _ => Ok(()), [INFO] [stdout] | ^ ...and 5 other patterns collectively make this unreachable [INFO] [stdout] = note: `#[warn(unreachable_patterns)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:87:21 [INFO] [stdout] | [INFO] [stdout] 87 | format!("new class").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"new class".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] = note: `#[warn(clippy::useless_format)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:93:21 [INFO] [stdout] | [INFO] [stdout] 93 | format!("get").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"get".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:99:21 [INFO] [stdout] | [INFO] [stdout] 99 | format!("set").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"set".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:104:17 [INFO] [stdout] | [INFO] [stdout] 104 | format!("this ") [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"this ".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:108:21 [INFO] [stdout] | [INFO] [stdout] 108 | format!("super").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"super".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `supper_class_expr` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:164:29 [INFO] [stdout] | [INFO] [stdout] 164 | if let Some(supper_class_expr) = supper_class { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_supper_class_expr` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `stmt` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:177:29 [INFO] [stdout] | [INFO] [stdout] 177 | fn visit_env(&mut self, stmt: &Stmt, env: Rc>) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^^ help: if this is intentional, prefix it with an underscore: `_stmt` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `env` [INFO] [stdout] --> src/vistor/stmt_resolver.rs:177:42 [INFO] [stdout] | [INFO] [stdout] 177 | fn visit_env(&mut self, stmt: &Stmt, env: Rc>) -> Result<(), ResolveError> { [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_env` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler [INFO] [stdout] --> src/vistor/expr_interpreter.rs:35:63 [INFO] [stdout] | [INFO] [stdout] 35 | let distance_option: Option<&usize> = self.locals.get(&name); [INFO] [stdout] | ^^^^^ help: change this to: `name` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow [INFO] [stdout] = note: `#[warn(clippy::needless_borrow)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:114:29 [INFO] [stdout] | [INFO] [stdout] 114 | ... return Ok(value); [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 114 - return Ok(value); [INFO] [stdout] 114 + Ok(value) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:118:42 [INFO] [stdout] | [INFO] [stdout] 118 | ... Ok(_) => return Ok(value), [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 118 - Ok(_) => return Ok(value), [INFO] [stdout] 118 + Ok(_) => Ok(value), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:119:45 [INFO] [stdout] | [INFO] [stdout] 119 | ... Err(err) => return Err(err), [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 119 - Err(err) => return Err(err), [INFO] [stdout] 119 + Err(err) => Err(err), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:141:29 [INFO] [stdout] | [INFO] [stdout] 141 | ... return Ok(LValue::Bool(true)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 141 - return Ok(LValue::Bool(true)); [INFO] [stdout] 141 + Ok(LValue::Bool(true)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:143:29 [INFO] [stdout] | [INFO] [stdout] 143 | ... return Ok(LValue::Bool(false)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 143 - return Ok(LValue::Bool(false)); [INFO] [stdout] 143 + Ok(LValue::Bool(false)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:154:29 [INFO] [stdout] | [INFO] [stdout] 154 | ... return Ok(LValue::Bool(true)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 154 - return Ok(LValue::Bool(true)); [INFO] [stdout] 154 + Ok(LValue::Bool(true)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:156:29 [INFO] [stdout] | [INFO] [stdout] 156 | ... return Ok(LValue::Bool(false)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 156 - return Ok(LValue::Bool(false)); [INFO] [stdout] 156 + Ok(LValue::Bool(false)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:177:25 [INFO] [stdout] | [INFO] [stdout] 177 | return function.call(self, paren, arguments_val); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 177 - return function.call(self, paren, arguments_val); [INFO] [stdout] 177 + function.call(self, paren, arguments_val) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:180:25 [INFO] [stdout] | [INFO] [stdout] 180 | return class.call(self, paren, arguments_val); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 180 - return class.call(self, paren, arguments_val); [INFO] [stdout] 180 + class.call(self, paren, arguments_val) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:192:17 [INFO] [stdout] | [INFO] [stdout] 192 | return val; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 192 - return val; [INFO] [stdout] 192 + val [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:199:25 [INFO] [stdout] | [INFO] [stdout] 199 | return Ok(instance.clone().get(field)?); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 199 - return Ok(instance.clone().get(field)?); [INFO] [stdout] 199 + Ok(instance.clone().get(field)?) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:216:25 [INFO] [stdout] | [INFO] [stdout] 216 | return Ok(value); [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 216 - return Ok(value); [INFO] [stdout] 216 + Ok(value) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:249:21 [INFO] [stdout] | [INFO] [stdout] 249 | return Ok(LValue::Nil); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 249 - return Ok(LValue::Nil); [INFO] [stdout] 249 + Ok(LValue::Nil) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:251:21 [INFO] [stdout] | [INFO] [stdout] 251 | / return Err(RunTimeError::Error(CommonError { [INFO] [stdout] 252 | | token: Some(token.clone()), [INFO] [stdout] 253 | | message: String::from("No super class found"), [INFO] [stdout] 254 | | })) [INFO] [stdout] | |_______________________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 251 ~ Err(RunTimeError::Error(CommonError { [INFO] [stdout] 252 + token: Some(token.clone()), [INFO] [stdout] 253 + message: String::from("No super class found"), [INFO] [stdout] 254 + })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enclosing `Ok` and `?` operator are unneeded [INFO] [stdout] --> src/vistor/expr_interpreter.rs:199:32 [INFO] [stdout] | [INFO] [stdout] 199 | return Ok(instance.clone().get(field)?); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark [INFO] [stdout] = note: `#[warn(clippy::needless_question_mark)]` on by default [INFO] [stdout] help: remove the enclosing `Ok` and `?` operator [INFO] [stdout] | [INFO] [stdout] 199 - return Ok(instance.clone().get(field)?); [INFO] [stdout] 199 + return instance.clone().get(field); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `function_type` [INFO] [stdout] --> src/parser/parser_stmt.rs:60:28 [INFO] [stdout] | [INFO] [stdout] 60 | fn function(&mut self, function_type: FunctionType) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_function_type` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/parser/parser_stmt.rs:170:13 [INFO] [stdout] | [INFO] [stdout] 170 | let mut initializer: Option; [INFO] [stdout] | ----^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this let-binding has unit value [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:63:21 [INFO] [stdout] | [INFO] [stdout] 63 | let value_result = self.exectue(statement)?; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value [INFO] [stdout] = note: `#[warn(clippy::let_unit_value)]` on by default [INFO] [stdout] help: omit the `let` binding [INFO] [stdout] | [INFO] [stdout] 63 - let value_result = self.exectue(statement)?; [INFO] [stdout] 63 + self.exectue(statement)?; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: deref on an immutable reference [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:83:34 [INFO] [stdout] | [INFO] [stdout] 83 | self.exectue(&(*body)); [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#borrow_deref_ref [INFO] [stdout] = note: `#[warn(clippy::borrow_deref_ref)]` on by default [INFO] [stdout] help: if you would like to reborrow, try removing `&*` [INFO] [stdout] | [INFO] [stdout] 83 - self.exectue(&(*body)); [INFO] [stdout] 83 + self.exectue(body); [INFO] [stdout] | [INFO] [stdout] help: if you would like to deref, try using `&**` [INFO] [stdout] | [INFO] [stdout] 83 - self.exectue(&(*body)); [INFO] [stdout] 83 + self.exectue(&**body); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `token` [INFO] [stdout] --> src/value/function.rs:66:9 [INFO] [stdout] | [INFO] [stdout] 66 | token: &Token, [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_token` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `res` [INFO] [stdout] --> src/interpreter/mod.rs:29:17 [INFO] [stdout] | [INFO] [stdout] 29 | let res = self.exectue(&statement)?; [INFO] [stdout] | ^^^ help: if this is intentional, prefix it with an underscore: `_res` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/parser/parser_expr.rs:151:9 [INFO] [stdout] | [INFO] [stdout] 151 | return self.class_init(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 151 - return self.class_init(); [INFO] [stdout] 151 + self.class_init() [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/parser/parser_expr.rs:162:9 [INFO] [stdout] | [INFO] [stdout] 162 | return self.call(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 162 - return self.call(); [INFO] [stdout] 162 + self.call() [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variant `Logical` is never constructed [INFO] [stdout] --> src/grammer/expression.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 4 | pub enum Expr { [INFO] [stdout] | ---- variant in this enum [INFO] [stdout] ... [INFO] [stdout] 11 | Logical(Box, Token, Box), [INFO] [stdout] | ^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Expr` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `interpret_stmt_debug` is never used [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:15:12 [INFO] [stdout] | [INFO] [stdout] 14 | impl Interpreter { [INFO] [stdout] | ---------------- method in this implementation [INFO] [stdout] 15 | pub fn interpret_stmt_debug(&mut self, stmt: &Stmt) -> Result<(), RunTimeError> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `or` and `and` are never used [INFO] [stdout] --> src/parser/parser_expr.rs:37:8 [INFO] [stdout] | [INFO] [stdout] 5 | impl Parser { [INFO] [stdout] | ----------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 37 | fn or(&mut self) -> Result { [INFO] [stdout] | ^^ [INFO] [stdout] ... [INFO] [stdout] 49 | fn and(&mut self) -> Result { [INFO] [stdout] | ^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `arity` is never used [INFO] [stdout] --> src/value/function.rs:59:8 [INFO] [stdout] | [INFO] [stdout] 52 | pub trait Callable { [INFO] [stdout] | -------- method in this trait [INFO] [stdout] ... [INFO] [stdout] 59 | fn arity(&self) -> usize; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `to_string` is never used [INFO] [stdout] --> src/value/class.rs:40:12 [INFO] [stdout] | [INFO] [stdout] 27 | impl Class { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 40 | pub fn to_string(&self) -> String { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `arity` is never used [INFO] [stdout] --> src/value/class.rs:77:12 [INFO] [stdout] | [INFO] [stdout] 61 | impl Class { [INFO] [stdout] | ---------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 77 | pub fn arity(&self) -> usize { [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `to_string` is never used [INFO] [stdout] --> src/value/class.rs:124:12 [INFO] [stdout] | [INFO] [stdout] 95 | impl ClassInstance { [INFO] [stdout] | ------------------ method in this implementation [INFO] [stdout] ... [INFO] [stdout] 124 | pub fn to_string(&self) -> String { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: using `write!()` with a format string that ends in a single newline [INFO] [stdout] --> src/scanner/token.rs:100:9 [INFO] [stdout] | [INFO] [stdout] 100 | / write!( [INFO] [stdout] 101 | | f, [INFO] [stdout] 102 | | "Token {{ type: {:?}, lexeme: \"{}\", literal: {:#?}, line: {:#?}, col: {:#?}}}\n", [INFO] [stdout] 103 | | self.typee, [INFO] [stdout] ... | [INFO] [stdout] 107 | | self.col, [INFO] [stdout] 108 | | ) [INFO] [stdout] | |_________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#write_with_newline [INFO] [stdout] = note: `#[warn(clippy::write_with_newline)]` on by default [INFO] [stdout] help: use `writeln!` instead [INFO] [stdout] | [INFO] [stdout] 100 ~ writeln!( [INFO] [stdout] 101 | f, [INFO] [stdout] 102 ~ "Token {{ type: {:?}, lexeme: \"{}\", literal: {:#?}, line: {:#?}, col: {:#?}}}", [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this boolean expression can be simplified [INFO] [stdout] --> src/parser/parser_stmt.rs:212:12 [INFO] [stdout] | [INFO] [stdout] 212 | if !increment.is_none() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: try: `increment.is_some()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stdout] = note: `#[warn(clippy::nonminimal_bool)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this boolean expression can be simplified [INFO] [stdout] --> src/parser/parser_stmt.rs:222:12 [INFO] [stdout] | [INFO] [stdout] 222 | if !initializer.is_none() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ help: try: `initializer.is_some()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: called `unwrap` on `condition` after checking its variant with `is_none` [INFO] [stdout] --> src/parser/parser_stmt.rs:208:31 [INFO] [stdout] | [INFO] [stdout] 205 | if condition.is_none() { [INFO] [stdout] | ---------------------- help: try: `if let Some() = condition` [INFO] [stdout] ... [INFO] [stdout] 208 | while_condition = condition.unwrap(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap [INFO] [stdout] = note: `#[warn(clippy::unnecessary_unwrap)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: called `unwrap` on `increment` after checking its variant with `is_none` [INFO] [stdout] --> src/parser/parser_stmt.rs:213:60 [INFO] [stdout] | [INFO] [stdout] 212 | if !increment.is_none() { [INFO] [stdout] | ------------------- the check is happening here [INFO] [stdout] 213 | while_body = Stmt::Block(vec![body, Stmt::Expr(increment.unwrap())]); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: try using `match` [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: called `unwrap` on `initializer` after checking its variant with `is_none` [INFO] [stdout] --> src/parser/parser_stmt.rs:223:43 [INFO] [stdout] | [INFO] [stdout] 222 | if !initializer.is_none() { [INFO] [stdout] | --------------------- the check is happening here [INFO] [stdout] 223 | while_body = Stmt::Block(vec![initializer.unwrap(), while_body]); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: try using `match` [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded late initialization [INFO] [stdout] --> src/parser/parser_stmt.rs:202:9 [INFO] [stdout] | [INFO] [stdout] 202 | let while_condition: Expr; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_late_init [INFO] [stdout] = note: `#[warn(clippy::needless_late_init)]` on by default [INFO] [stdout] help: move the declaration `while_condition` here and remove the assignments from the branches [INFO] [stdout] | [INFO] [stdout] 202 ~ [INFO] [stdout] 203 | [INFO] [stdout] 204 | // Create condition for while statement [INFO] [stdout] 205 ~ let while_condition: Expr = if condition.is_none() { [INFO] [stdout] 206 ~ Expr::Literal(ExprLiteral::True) [INFO] [stdout] 207 | } else { [INFO] [stdout] 208 ~ condition.unwrap() [INFO] [stdout] 209 ~ }; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual `RangeInclusive::contains` implementation [INFO] [stdout] --> src/scanner/mod.rs:257:9 [INFO] [stdout] | [INFO] [stdout] 257 | c >= '0' && c <= '9' [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: use: `('0'..='9').contains(&c)` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains [INFO] [stdout] = note: `#[warn(clippy::manual_range_contains)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual `RangeInclusive::contains` implementation [INFO] [stdout] --> src/scanner/mod.rs:261:9 [INFO] [stdout] | [INFO] [stdout] 261 | (c >= 'a' && c <= 'z') || [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ help: use: `('a'..='z').contains(&c)` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: manual `RangeInclusive::contains` implementation [INFO] [stdout] --> src/scanner/mod.rs:262:9 [INFO] [stdout] | [INFO] [stdout] 262 | (c >= 'A' && c <= 'Z') || [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ help: use: `('A'..='Z').contains(&c)` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/scanner/mod.rs:267:9 [INFO] [stdout] | [INFO] [stdout] 267 | return Scanner::is_alpha(c) || Scanner::is_digit(c) [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] = note: `#[warn(clippy::needless_return)]` on by default [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 267 - return Scanner::is_alpha(c) || Scanner::is_digit(c) [INFO] [stdout] 267 + Scanner::is_alpha(c) || Scanner::is_digit(c) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/function.rs:112:9 [INFO] [stdout] | [INFO] [stdout] 112 | return 0; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 112 - return 0; [INFO] [stdout] 112 + 0 [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: implementation of inherent method `to_string(&self) -> String` for type `value::class::Class` [INFO] [stdout] --> src/value/class.rs:40:5 [INFO] [stdout] | [INFO] [stdout] 40 | / pub fn to_string(&self) -> String { [INFO] [stdout] 41 | | self.name.clone() [INFO] [stdout] 42 | | } [INFO] [stdout] | |_____^ [INFO] [stdout] | [INFO] [stdout] = help: implement trait `Display` for type `value::class::Class` instead [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#inherent_to_string [INFO] [stdout] = note: `#[warn(clippy::inherent_to_string)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/class.rs:57:9 [INFO] [stdout] | [INFO] [stdout] 57 | return None; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 57 - return None; [INFO] [stdout] 57 + None [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/class.rs:78:9 [INFO] [stdout] | [INFO] [stdout] 78 | return 0; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 78 - return 0; [INFO] [stdout] 78 + 0 [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/class.rs:121:9 [INFO] [stdout] | [INFO] [stdout] 121 | return instance; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 121 - return instance; [INFO] [stdout] 121 + instance [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: implementation of inherent method `to_string(&self) -> String` for type `value::class::ClassInstance` [INFO] [stdout] --> src/value/class.rs:124:5 [INFO] [stdout] | [INFO] [stdout] 124 | / pub fn to_string(&self) -> String { [INFO] [stdout] 125 | | format!("{} instance", self.class.name) [INFO] [stdout] 126 | | } [INFO] [stdout] | |_____^ [INFO] [stdout] | [INFO] [stdout] = help: implement trait `Display` for type `value::class::ClassInstance` instead [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#inherent_to_string [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this let-binding has unit value [INFO] [stdout] --> src/interpreter/mod.rs:29:13 [INFO] [stdout] | [INFO] [stdout] 29 | let res = self.exectue(&statement)?; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value [INFO] [stdout] help: omit the `let` binding [INFO] [stdout] | [INFO] [stdout] 29 - let res = self.exectue(&statement)?; [INFO] [stdout] 29 + self.exectue(&statement)?; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: usage of `contains_key` followed by `insert` on a `HashMap` [INFO] [stdout] --> src/environment/mod.rs:31:9 [INFO] [stdout] | [INFO] [stdout] 31 | / if self.map.contains_key(&name_string) { [INFO] [stdout] 32 | | self.map.insert(name_string, value); [INFO] [stdout] 33 | | return Ok(()); [INFO] [stdout] 34 | | } [INFO] [stdout] | |_________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_entry [INFO] [stdout] = note: `#[warn(clippy::map_entry)]` on by default [INFO] [stdout] help: try [INFO] [stdout] | [INFO] [stdout] 31 ~ if let std::collections::hash_map::Entry::Occupied(mut e) = self.map.entry(name_string) { [INFO] [stdout] 32 + e.insert(value); [INFO] [stdout] 33 + return Ok(()); [INFO] [stdout] 34 + } [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calling `push_str()` using a single-character string literal [INFO] [stdout] --> src/vistor/ast_printer.rs:20:9 [INFO] [stdout] | [INFO] [stdout] 20 | output.push_str("("); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `output.push('(')` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str [INFO] [stdout] = note: `#[warn(clippy::single_char_add_str)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calling `push_str()` using a single-character string literal [INFO] [stdout] --> src/vistor/ast_printer.rs:24:13 [INFO] [stdout] | [INFO] [stdout] 24 | output.push_str(" "); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `output.push(' ')` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calling `push_str()` using a single-character string literal [INFO] [stdout] --> src/vistor/ast_printer.rs:28:9 [INFO] [stdout] | [INFO] [stdout] 28 | output.push_str(")"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `output.push(')')` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `push` immediately after creation [INFO] [stdout] --> src/resolver/mod.rs:39:9 [INFO] [stdout] | [INFO] [stdout] 39 | / let mut init_scopes: Vec> = Vec::new(); [INFO] [stdout] 40 | | init_scopes.push(HashMap::new()); [INFO] [stdout] | |_________________________________________^ help: consider using the `vec![]` macro: `let init_scopes: Vec> = vec![..];` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#vec_init_then_push [INFO] [stdout] = note: `#[warn(clippy::vec_init_then_push)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:87:21 [INFO] [stdout] | [INFO] [stdout] 87 | format!("new class").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"new class".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] = note: `#[warn(clippy::useless_format)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:93:21 [INFO] [stdout] | [INFO] [stdout] 93 | format!("get").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"get".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:99:21 [INFO] [stdout] | [INFO] [stdout] 99 | format!("set").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"set".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:104:17 [INFO] [stdout] | [INFO] [stdout] 104 | format!("this ") [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"this ".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: useless use of `format!` [INFO] [stdout] --> src/vistor/ast_printer.rs:108:21 [INFO] [stdout] | [INFO] [stdout] 108 | format!("super").as_str(), [INFO] [stdout] | ^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `"super".to_string()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this expression creates a reference which is immediately dereferenced by the compiler [INFO] [stdout] --> src/vistor/expr_interpreter.rs:35:63 [INFO] [stdout] | [INFO] [stdout] 35 | let distance_option: Option<&usize> = self.locals.get(&name); [INFO] [stdout] | ^^^^^ help: change this to: `name` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow [INFO] [stdout] = note: `#[warn(clippy::needless_borrow)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:114:29 [INFO] [stdout] | [INFO] [stdout] 114 | ... return Ok(value); [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 114 - return Ok(value); [INFO] [stdout] 114 + Ok(value) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:118:42 [INFO] [stdout] | [INFO] [stdout] 118 | ... Ok(_) => return Ok(value), [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 118 - Ok(_) => return Ok(value), [INFO] [stdout] 118 + Ok(_) => Ok(value), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:119:45 [INFO] [stdout] | [INFO] [stdout] 119 | ... Err(err) => return Err(err), [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 119 - Err(err) => return Err(err), [INFO] [stdout] 119 + Err(err) => Err(err), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:141:29 [INFO] [stdout] | [INFO] [stdout] 141 | ... return Ok(LValue::Bool(true)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 141 - return Ok(LValue::Bool(true)); [INFO] [stdout] 141 + Ok(LValue::Bool(true)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:143:29 [INFO] [stdout] | [INFO] [stdout] 143 | ... return Ok(LValue::Bool(false)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 143 - return Ok(LValue::Bool(false)); [INFO] [stdout] 143 + Ok(LValue::Bool(false)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:154:29 [INFO] [stdout] | [INFO] [stdout] 154 | ... return Ok(LValue::Bool(true)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 154 - return Ok(LValue::Bool(true)); [INFO] [stdout] 154 + Ok(LValue::Bool(true)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:156:29 [INFO] [stdout] | [INFO] [stdout] 156 | ... return Ok(LValue::Bool(false)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 156 - return Ok(LValue::Bool(false)); [INFO] [stdout] 156 + Ok(LValue::Bool(false)) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:177:25 [INFO] [stdout] | [INFO] [stdout] 177 | return function.call(self, paren, arguments_val); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 177 - return function.call(self, paren, arguments_val); [INFO] [stdout] 177 + function.call(self, paren, arguments_val) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:180:25 [INFO] [stdout] | [INFO] [stdout] 180 | return class.call(self, paren, arguments_val); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 180 - return class.call(self, paren, arguments_val); [INFO] [stdout] 180 + class.call(self, paren, arguments_val) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:192:17 [INFO] [stdout] | [INFO] [stdout] 192 | return val; [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 192 - return val; [INFO] [stdout] 192 + val [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:83:21 [INFO] [stdout] | [INFO] [stdout] 83 | self.exectue(&(*body)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 83 | let _ = self.exectue(&(*body)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:199:25 [INFO] [stdout] | [INFO] [stdout] 199 | return Ok(instance.clone().get(field)?); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 199 - return Ok(instance.clone().get(field)?); [INFO] [stdout] 199 + Ok(instance.clone().get(field)?) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:216:25 [INFO] [stdout] | [INFO] [stdout] 216 | return Ok(value); [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 216 - return Ok(value); [INFO] [stdout] 216 + Ok(value) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:249:21 [INFO] [stdout] | [INFO] [stdout] 249 | return Ok(LValue::Nil); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 249 - return Ok(LValue::Nil); [INFO] [stdout] 249 + Ok(LValue::Nil) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/vistor/expr_interpreter.rs:251:21 [INFO] [stdout] | [INFO] [stdout] 251 | / return Err(RunTimeError::Error(CommonError { [INFO] [stdout] 252 | | token: Some(token.clone()), [INFO] [stdout] 253 | | message: String::from("No super class found"), [INFO] [stdout] 254 | | })) [INFO] [stdout] | |_______________________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 251 ~ Err(RunTimeError::Error(CommonError { [INFO] [stdout] 252 + token: Some(token.clone()), [INFO] [stdout] 253 + message: String::from("No super class found"), [INFO] [stdout] 254 + })) [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:42:13 [INFO] [stdout] | [INFO] [stdout] 42 | self.consume(TokenType::Identifier, "Expect super class name"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 42 | let _ = self.consume(TokenType::Identifier, "Expect super class name"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:125:9 [INFO] [stdout] | [INFO] [stdout] 125 | self.consume(TokenType::RightBrace, "Expect '}' after block."); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 125 | let _ = self.consume(TokenType::RightBrace, "Expect '}' after block."); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:143:9 [INFO] [stdout] | [INFO] [stdout] 143 | self.consume(TokenType::LeftParen, "Expect \' ( \' after if"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 143 | let _ = self.consume(TokenType::LeftParen, "Expect \' ( \' after if"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:145:9 [INFO] [stdout] | [INFO] [stdout] 145 | self.consume(TokenType::RightParen, "Expect \' ) \' after if condition"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 145 | let _ = self.consume(TokenType::RightParen, "Expect \' ) \' after if condition"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:158:9 [INFO] [stdout] | [INFO] [stdout] 158 | self.consume(TokenType::LeftParen, "Expect \' ( \' after while"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 158 | let _ = self.consume(TokenType::LeftParen, "Expect \' ( \' after while"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:160:9 [INFO] [stdout] | [INFO] [stdout] 160 | self.consume(TokenType::RightParen, "Expect \' ) \' after condition"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 160 | let _ = self.consume(TokenType::RightParen, "Expect \' ) \' after condition"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:167:9 [INFO] [stdout] | [INFO] [stdout] 167 | self.consume(TokenType::LeftParen, "Expect \' ( \' after for"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 167 | let _ = self.consume(TokenType::LeftParen, "Expect \' ( \' after for"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:186:9 [INFO] [stdout] | [INFO] [stdout] 186 | self.consume(TokenType::Semicolon, "Expect \' ; \' after loop condition"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 186 | let _ = self.consume(TokenType::Semicolon, "Expect \' ; \' after loop condition"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:195:9 [INFO] [stdout] | [INFO] [stdout] 195 | self.consume(TokenType::RightParen, "Expect \' ) \' after for clauses."); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 195 | let _ = self.consume(TokenType::RightParen, "Expect \' ) \' after for clauses."); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:237:9 [INFO] [stdout] | [INFO] [stdout] 237 | self.consume(TokenType::Semicolon, "Expect \';\' after return value"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 237 | let _ = self.consume(TokenType::Semicolon, "Expect \';\' after return value"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/value/class.rs:118:13 [INFO] [stdout] | [INFO] [stdout] 118 | init_func.bind(instance.clone()).call(interpreter, token, arguments); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 118 | let _ = init_func.bind(instance.clone()).call(interpreter, token, arguments); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's named elsewhere is confusing [INFO] [stdout] --> src/resolver/mod.rs:38:53 [INFO] [stdout] | [INFO] [stdout] 38 | pub fn new(interpreter: &'a mut Interpreter) -> Resolver { [INFO] [stdout] | -- ^^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is named here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: consistently use `'a` [INFO] [stdout] | [INFO] [stdout] 38 | pub fn new(interpreter: &'a mut Interpreter) -> Resolver<'a> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: enclosing `Ok` and `?` operator are unneeded [INFO] [stdout] --> src/vistor/expr_interpreter.rs:199:32 [INFO] [stdout] | [INFO] [stdout] 199 | return Ok(instance.clone().get(field)?); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark [INFO] [stdout] = note: `#[warn(clippy::needless_question_mark)]` on by default [INFO] [stdout] help: remove the enclosing `Ok` and `?` operator [INFO] [stdout] | [INFO] [stdout] 199 - return Ok(instance.clone().get(field)?); [INFO] [stdout] 199 + return instance.clone().get(field); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this let-binding has unit value [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:63:21 [INFO] [stdout] | [INFO] [stdout] 63 | let value_result = self.exectue(statement)?; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value [INFO] [stdout] = note: `#[warn(clippy::let_unit_value)]` on by default [INFO] [stdout] help: omit the `let` binding [INFO] [stdout] | [INFO] [stdout] 63 - let value_result = self.exectue(statement)?; [INFO] [stdout] 63 + self.exectue(statement)?; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: deref on an immutable reference [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:83:34 [INFO] [stdout] | [INFO] [stdout] 83 | self.exectue(&(*body)); [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#borrow_deref_ref [INFO] [stdout] = note: `#[warn(clippy::borrow_deref_ref)]` on by default [INFO] [stdout] help: if you would like to reborrow, try removing `&*` [INFO] [stdout] | [INFO] [stdout] 83 - self.exectue(&(*body)); [INFO] [stdout] 83 + self.exectue(body); [INFO] [stdout] | [INFO] [stdout] help: if you would like to deref, try using `&**` [INFO] [stdout] | [INFO] [stdout] 83 - self.exectue(&(*body)); [INFO] [stdout] 83 + self.exectue(&**body); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/parser/parser_expr.rs:151:9 [INFO] [stdout] | [INFO] [stdout] 151 | return self.class_init(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 151 - return self.class_init(); [INFO] [stdout] 151 + self.class_init() [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/parser/parser_expr.rs:162:9 [INFO] [stdout] | [INFO] [stdout] 162 | return self.call(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 162 - return self.call(); [INFO] [stdout] 162 + self.call() [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this boolean expression can be simplified [INFO] [stdout] --> src/parser/parser_stmt.rs:212:12 [INFO] [stdout] | [INFO] [stdout] 212 | if !increment.is_none() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ help: try: `increment.is_some()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stdout] = note: `#[warn(clippy::nonminimal_bool)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this boolean expression can be simplified [INFO] [stdout] --> src/parser/parser_stmt.rs:222:12 [INFO] [stdout] | [INFO] [stdout] 222 | if !initializer.is_none() { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ help: try: `initializer.is_some()` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#nonminimal_bool [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: called `unwrap` on `condition` after checking its variant with `is_none` [INFO] [stdout] --> src/parser/parser_stmt.rs:208:31 [INFO] [stdout] | [INFO] [stdout] 205 | if condition.is_none() { [INFO] [stdout] | ---------------------- help: try: `if let Some() = condition` [INFO] [stdout] ... [INFO] [stdout] 208 | while_condition = condition.unwrap(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap [INFO] [stdout] = note: `#[warn(clippy::unnecessary_unwrap)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: called `unwrap` on `increment` after checking its variant with `is_none` [INFO] [stdout] --> src/parser/parser_stmt.rs:213:60 [INFO] [stdout] | [INFO] [stdout] 212 | if !increment.is_none() { [INFO] [stdout] | ------------------- the check is happening here [INFO] [stdout] 213 | while_body = Stmt::Block(vec![body, Stmt::Expr(increment.unwrap())]); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: try using `match` [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: called `unwrap` on `initializer` after checking its variant with `is_none` [INFO] [stdout] --> src/parser/parser_stmt.rs:223:43 [INFO] [stdout] | [INFO] [stdout] 222 | if !initializer.is_none() { [INFO] [stdout] | --------------------- the check is happening here [INFO] [stdout] 223 | while_body = Stmt::Block(vec![initializer.unwrap(), while_body]); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: try using `match` [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded late initialization [INFO] [stdout] --> src/parser/parser_stmt.rs:202:9 [INFO] [stdout] | [INFO] [stdout] 202 | let while_condition: Expr; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_late_init [INFO] [stdout] = note: `#[warn(clippy::needless_late_init)]` on by default [INFO] [stdout] help: move the declaration `while_condition` here and remove the assignments from the branches [INFO] [stdout] | [INFO] [stdout] 202 ~ [INFO] [stdout] 203 | [INFO] [stdout] 204 | // Create condition for while statement [INFO] [stdout] 205 ~ let while_condition: Expr = if condition.is_none() { [INFO] [stdout] 206 ~ Expr::Literal(ExprLiteral::True) [INFO] [stdout] 207 | } else { [INFO] [stdout] 208 ~ condition.unwrap() [INFO] [stdout] 209 ~ }; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/function.rs:112:9 [INFO] [stdout] | [INFO] [stdout] 112 | return 0; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 112 - return 0; [INFO] [stdout] 112 + 0 [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: implementation of inherent method `to_string(&self) -> String` for type `value::class::Class` [INFO] [stdout] --> src/value/class.rs:40:5 [INFO] [stdout] | [INFO] [stdout] 40 | / pub fn to_string(&self) -> String { [INFO] [stdout] 41 | | self.name.clone() [INFO] [stdout] 42 | | } [INFO] [stdout] | |_____^ [INFO] [stdout] | [INFO] [stdout] = help: implement trait `Display` for type `value::class::Class` instead [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#inherent_to_string [INFO] [stdout] = note: `#[warn(clippy::inherent_to_string)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/class.rs:57:9 [INFO] [stdout] | [INFO] [stdout] 57 | return None; [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 57 - return None; [INFO] [stdout] 57 + None [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/class.rs:78:9 [INFO] [stdout] | [INFO] [stdout] 78 | return 0; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 78 - return 0; [INFO] [stdout] 78 + 0 [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unneeded `return` statement [INFO] [stdout] --> src/value/class.rs:121:9 [INFO] [stdout] | [INFO] [stdout] 121 | return instance; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stdout] help: remove `return` [INFO] [stdout] | [INFO] [stdout] 121 - return instance; [INFO] [stdout] 121 + instance [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: implementation of inherent method `to_string(&self) -> String` for type `value::class::ClassInstance` [INFO] [stdout] --> src/value/class.rs:124:5 [INFO] [stdout] | [INFO] [stdout] 124 | / pub fn to_string(&self) -> String { [INFO] [stdout] 125 | | format!("{} instance", self.class.name) [INFO] [stdout] 126 | | } [INFO] [stdout] | |_____^ [INFO] [stdout] | [INFO] [stdout] = help: implement trait `Display` for type `value::class::ClassInstance` instead [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#inherent_to_string [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: this let-binding has unit value [INFO] [stdout] --> src/interpreter/mod.rs:29:13 [INFO] [stdout] | [INFO] [stdout] 29 | let res = self.exectue(&statement)?; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value [INFO] [stdout] help: omit the `let` binding [INFO] [stdout] | [INFO] [stdout] 29 - let res = self.exectue(&statement)?; [INFO] [stdout] 29 + self.exectue(&statement)?; [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: usage of `contains_key` followed by `insert` on a `HashMap` [INFO] [stdout] --> src/environment/mod.rs:31:9 [INFO] [stdout] | [INFO] [stdout] 31 | / if self.map.contains_key(&name_string) { [INFO] [stdout] 32 | | self.map.insert(name_string, value); [INFO] [stdout] 33 | | return Ok(()); [INFO] [stdout] 34 | | } [INFO] [stdout] | |_________^ [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_entry [INFO] [stdout] = note: `#[warn(clippy::map_entry)]` on by default [INFO] [stdout] help: try [INFO] [stdout] | [INFO] [stdout] 31 ~ if let std::collections::hash_map::Entry::Occupied(mut e) = self.map.entry(name_string) { [INFO] [stdout] 32 + e.insert(value); [INFO] [stdout] 33 + return Ok(()); [INFO] [stdout] 34 + } [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: calls to `push` immediately after creation [INFO] [stdout] --> src/resolver/mod.rs:39:9 [INFO] [stdout] | [INFO] [stdout] 39 | / let mut init_scopes: Vec> = Vec::new(); [INFO] [stdout] 40 | | init_scopes.push(HashMap::new()); [INFO] [stdout] | |_________________________________________^ help: consider using the `vec![]` macro: `let init_scopes: Vec> = vec![..];` [INFO] [stdout] | [INFO] [stdout] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#vec_init_then_push [INFO] [stdout] = note: `#[warn(clippy::vec_init_then_push)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/vistor/stmt_interpreter.rs:83:21 [INFO] [stdout] | [INFO] [stdout] 83 | self.exectue(&(*body)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 83 | let _ = self.exectue(&(*body)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:42:13 [INFO] [stdout] | [INFO] [stdout] 42 | self.consume(TokenType::Identifier, "Expect super class name"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 42 | let _ = self.consume(TokenType::Identifier, "Expect super class name"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:125:9 [INFO] [stdout] | [INFO] [stdout] 125 | self.consume(TokenType::RightBrace, "Expect '}' after block."); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 125 | let _ = self.consume(TokenType::RightBrace, "Expect '}' after block."); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:143:9 [INFO] [stdout] | [INFO] [stdout] 143 | self.consume(TokenType::LeftParen, "Expect \' ( \' after if"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 143 | let _ = self.consume(TokenType::LeftParen, "Expect \' ( \' after if"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:145:9 [INFO] [stdout] | [INFO] [stdout] 145 | self.consume(TokenType::RightParen, "Expect \' ) \' after if condition"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 145 | let _ = self.consume(TokenType::RightParen, "Expect \' ) \' after if condition"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:158:9 [INFO] [stdout] | [INFO] [stdout] 158 | self.consume(TokenType::LeftParen, "Expect \' ( \' after while"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 158 | let _ = self.consume(TokenType::LeftParen, "Expect \' ( \' after while"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:160:9 [INFO] [stdout] | [INFO] [stdout] 160 | self.consume(TokenType::RightParen, "Expect \' ) \' after condition"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 160 | let _ = self.consume(TokenType::RightParen, "Expect \' ) \' after condition"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:167:9 [INFO] [stdout] | [INFO] [stdout] 167 | self.consume(TokenType::LeftParen, "Expect \' ( \' after for"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 167 | let _ = self.consume(TokenType::LeftParen, "Expect \' ( \' after for"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:186:9 [INFO] [stdout] | [INFO] [stdout] 186 | self.consume(TokenType::Semicolon, "Expect \' ; \' after loop condition"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 186 | let _ = self.consume(TokenType::Semicolon, "Expect \' ; \' after loop condition"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:195:9 [INFO] [stdout] | [INFO] [stdout] 195 | self.consume(TokenType::RightParen, "Expect \' ) \' after for clauses."); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 195 | let _ = self.consume(TokenType::RightParen, "Expect \' ) \' after for clauses."); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/parser/parser_stmt.rs:237:9 [INFO] [stdout] | [INFO] [stdout] 237 | self.consume(TokenType::Semicolon, "Expect \';\' after return value"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 237 | let _ = self.consume(TokenType::Semicolon, "Expect \';\' after return value"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `std::result::Result` that must be used [INFO] [stdout] --> src/value/class.rs:118:13 [INFO] [stdout] | [INFO] [stdout] 118 | init_func.bind(instance.clone()).call(interpreter, token, arguments); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 118 | let _ = init_func.bind(instance.clone()).call(interpreter, token, arguments); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's named elsewhere is confusing [INFO] [stdout] --> src/resolver/mod.rs:38:53 [INFO] [stdout] | [INFO] [stdout] 38 | pub fn new(interpreter: &'a mut Interpreter) -> Resolver { [INFO] [stdout] | -- ^^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is named here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: consistently use `'a` [INFO] [stdout] | [INFO] [stdout] 38 | pub fn new(interpreter: &'a mut Interpreter) -> Resolver<'a> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 6.65s [INFO] running `Command { std: "docker" "inspect" "d95fc17780d21420c1828570598334e49d3d4d5b201e31fc1702346f6924794e", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d95fc17780d21420c1828570598334e49d3d4d5b201e31fc1702346f6924794e", kill_on_drop: false }` [INFO] [stdout] d95fc17780d21420c1828570598334e49d3d4d5b201e31fc1702346f6924794e