[INFO] cloning repository https://github.com/archerfeel/azeroth-vm [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/archerfeel/azeroth-vm" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Farcherfeel%2Fazeroth-vm", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Farcherfeel%2Fazeroth-vm'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 30a4cbabf144cec914b5b5aa15bfa78d34da6af4 [INFO] checking archerfeel/azeroth-vm against master#32cd9114712a24010b0583624dc52ac302194128 for pr-143717 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Farcherfeel%2Fazeroth-vm" "/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/archerfeel/azeroth-vm [INFO] finished tweaking git repo https://github.com/archerfeel/azeroth-vm [INFO] tweaked toml for git repo https://github.com/archerfeel/azeroth-vm written to /workspace/builds/worker-7-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/archerfeel/azeroth-vm on toolchain 32cd9114712a24010b0583624dc52ac302194128 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+32cd9114712a24010b0583624dc52ac302194128" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+32cd9114712a24010b0583624dc52ac302194128" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Updating crates.io index [INFO] [stderr] Blocking waiting for file lock on package cache [INFO] [stderr] Locking 44 packages to latest compatible versions [INFO] [stderr] Adding base64 v0.11.0 (available: v0.22.1) [INFO] [stderr] Adding libloading v0.5.2 (available: v0.8.8) [INFO] [stderr] Adding regex v0.2.11 (available: v1.11.1) [INFO] [stderr] Adding wasi v0.10.0+wasi-snapshot-preview1 (available: v0.10.2+wasi-snapshot-preview1) [INFO] [stderr] Adding zip v0.3.3 (available: v4.3.0) [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+32cd9114712a24010b0583624dc52ac302194128" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded podio v0.1.7 [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:90999bfc7ae267e83380e433d8e61a7c072ca6729e92edbae886d3423b3a6f4c" "/opt/rustwide/cargo-home/bin/cargo" "+32cd9114712a24010b0583624dc52ac302194128" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 7ceb212188d1b7d0a7429baf7901d511c6cfaaf84877c4ae5fb53df9e65c49ef [INFO] running `Command { std: "docker" "start" "-a" "7ceb212188d1b7d0a7429baf7901d511c6cfaaf84877c4ae5fb53df9e65c49ef", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "7ceb212188d1b7d0a7429baf7901d511c6cfaaf84877c4ae5fb53df9e65c49ef", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "7ceb212188d1b7d0a7429baf7901d511c6cfaaf84877c4ae5fb53df9e65c49ef", kill_on_drop: false }` [INFO] [stdout] 7ceb212188d1b7d0a7429baf7901d511c6cfaaf84877c4ae5fb53df9e65c49ef [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:90999bfc7ae267e83380e433d8e61a7c072ca6729e92edbae886d3423b3a6f4c" "/opt/rustwide/cargo-home/bin/cargo" "+32cd9114712a24010b0583624dc52ac302194128" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 5ef55410bb0c377665a50ac56ec13817ffee1148611dbd2f7626743706b7dfb0 [INFO] running `Command { std: "docker" "start" "-a" "5ef55410bb0c377665a50ac56ec13817ffee1148611dbd2f7626743706b7dfb0", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.2.174 [INFO] [stderr] Compiling shlex v1.3.0 [INFO] [stderr] Compiling maybe-uninit v2.0.0 [INFO] [stderr] Compiling pkg-config v0.3.32 [INFO] [stderr] Checking cfg-if v1.0.1 [INFO] [stderr] Checking stable_deref_trait v1.2.0 [INFO] [stderr] Checking adler2 v2.0.1 [INFO] [stderr] Checking memchr v2.7.5 [INFO] [stderr] Checking lazy_static v1.5.0 [INFO] [stderr] Compiling regex v0.2.11 [INFO] [stderr] Checking ucd-util v0.1.10 [INFO] [stderr] Checking podio v0.1.7 [INFO] [stderr] Checking utf8-ranges v1.0.5 [INFO] [stderr] Checking base64 v0.11.0 [INFO] [stderr] Checking azeroth-jni v0.1.0 (/opt/rustwide/workdir/jni) [INFO] [stderr] Checking log v0.4.27 [INFO] [stderr] Checking thread_local v0.3.6 [INFO] [stderr] Checking crc32fast v1.4.2 [INFO] [stderr] Checking owning_ref v0.3.3 [INFO] [stderr] Checking argparse v0.2.2 [INFO] [stderr] Checking miniz_oxide v0.8.9 [INFO] [stderr] Compiling cc v1.2.29 [INFO] [stderr] Checking regex-syntax v0.5.6 [INFO] [stderr] Checking smallvec v0.6.14 [INFO] [stderr] Checking flate2 v1.1.2 [INFO] [stderr] Checking aho-corasick v0.6.10 [INFO] [stderr] Checking rand v0.4.6 [INFO] [stderr] Checking time v0.1.45 [INFO] [stderr] Checking msdos_time v0.1.6 [INFO] [stderr] Compiling bzip2-sys v0.1.13+1.0.8 [INFO] [stderr] Compiling libloading v0.5.2 [INFO] [stderr] Checking parking_lot_core v0.2.14 [INFO] [stderr] Checking parking_lot v0.4.8 [INFO] [stderr] Checking chashmap v2.2.2 [INFO] [stderr] Checking bzip2 v0.3.3 [INFO] [stderr] Checking zip v0.3.3 [INFO] [stderr] Checking azeroth-vm v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `crate::gc` [INFO] [stdout] --> src/mem/heap.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::gc; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::thread::Thread` [INFO] [stdout] --> src/interpreter/mod.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 11 | use std::thread::Thread; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::gc` [INFO] [stdout] --> src/mem/heap.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::gc; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::thread::Thread` [INFO] [stdout] --> src/interpreter/mod.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 11 | use std::thread::Thread; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the feature `map_first_last` has been stable since 1.66.0 and no longer requires an attribute to enable [INFO] [stdout] --> src/lib.rs:2:12 [INFO] [stdout] | [INFO] [stdout] 2 | #![feature(map_first_last)] [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(stable_features)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the feature `weak_into_raw` has been stable since 1.45.0 and no longer requires an attribute to enable [INFO] [stdout] --> src/lib.rs:1:12 [INFO] [stdout] | [INFO] [stdout] 1 | #![feature(weak_into_raw)] [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the feature `map_first_last` has been stable since 1.66.0 and no longer requires an attribute to enable [INFO] [stdout] --> src/lib.rs:2:12 [INFO] [stdout] | [INFO] [stdout] 2 | #![feature(map_first_last)] [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(stable_features)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: the feature `weak_into_raw` has been stable since 1.45.0 and no longer requires an attribute to enable [INFO] [stdout] --> src/lib.rs:1:12 [INFO] [stdout] | [INFO] [stdout] 1 | #![feature(weak_into_raw)] [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/mem/heap.rs:183:9 [INFO] [stdout] | [INFO] [stdout] 159 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] 160 | let eden_ptr = jvm_heap!().base.add(eden.offset as usize); [INFO] [stdout] | ----------- in this macro invocation [INFO] [stdout] ... [INFO] [stdout] 183 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` on by default [INFO] [stdout] = note: this warning originates in the macro `jvm_heap` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/mem/heap.rs:183:9 [INFO] [stdout] | [INFO] [stdout] 159 | unsafe { [INFO] [stdout] | ------ because it's nested under this `unsafe` block [INFO] [stdout] 160 | let eden_ptr = jvm_heap!().base.add(eden.offset as usize); [INFO] [stdout] | ----------- in this macro invocation [INFO] [stdout] ... [INFO] [stdout] 183 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_unsafe)]` on by default [INFO] [stdout] = note: this warning originates in the macro `jvm_heap` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/mem/heap.rs:183:9 [INFO] [stdout] | [INFO] [stdout] 170 | unsafe { jvm_heap!().base.add(offset) } [INFO] [stdout] | ------ ----------- in this macro invocation [INFO] [stdout] | | [INFO] [stdout] | because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 183 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `jvm_heap` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/mem/heap.rs:183:9 [INFO] [stdout] | [INFO] [stdout] 183 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 219 | let obj0_ptr = unsafe { jvm_heap!().base.add(obj0 as usize) }; [INFO] [stdout] | ------ ----------- in this macro invocation [INFO] [stdout] | | [INFO] [stdout] | because it's nested under this `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `jvm_heap` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary `unsafe` block [INFO] [stdout] --> src/mem/heap.rs:183:9 [INFO] [stdout] | [INFO] [stdout] 170 | unsafe { jvm_heap!().base.add(offset) } [INFO] [stdout] | ------ ----------- in this macro invocation [INFO] [stdout] | | [INFO] [stdout] | because it's nested under this `unsafe` block [INFO] [stdout] ... [INFO] [stdout] 183 | unsafe { [INFO] [stdout] | ^^^^^^ unnecessary `unsafe` block [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `jvm_heap` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `klass` [INFO] [stdout] --> src/interpreter/mod.rs:597:22 [INFO] [stdout] | [INFO] [stdout] 597 | let (klass, initialized) = found.unwrap(); [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_klass` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `klass` [INFO] [stdout] --> src/interpreter/mod.rs:597:22 [INFO] [stdout] | [INFO] [stdout] 597 | let (klass, initialized) = found.unwrap(); [INFO] [stdout] | ^^^^^ help: if this is intentional, prefix it with an underscore: `_klass` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `init_value` is never used [INFO] [stdout] --> src/bytecode/field.rs:93:4 [INFO] [stdout] | [INFO] [stdout] 93 | fn init_value(access_flag: u16, descriptor: &str) -> Option { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_PRIVATE` is never used [INFO] [stdout] --> src/bytecode/method.rs:9:7 [INFO] [stdout] | [INFO] [stdout] 9 | const ACC_PRIVATE: U2 = 0x0002; // Declared private; accessible only within the defining class. [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_BRIDGE` is never used [INFO] [stdout] --> src/bytecode/method.rs:19:7 [INFO] [stdout] | [INFO] [stdout] 19 | const ACC_BRIDGE: U2 = 0x0040; // A bridge method, generated by the compiler. [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_VARARGS` is never used [INFO] [stdout] --> src/bytecode/method.rs:21:7 [INFO] [stdout] | [INFO] [stdout] 21 | const ACC_VARARGS: U2 = 0x0080; // Declared with variable number of arguments. [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_ABSTRACT` is never used [INFO] [stdout] --> src/bytecode/method.rs:25:7 [INFO] [stdout] | [INFO] [stdout] 25 | const ACC_ABSTRACT: U2 = 0x0400; // Declared abstract; no implementation is provided. [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_STRICT` is never used [INFO] [stdout] --> src/bytecode/method.rs:27:7 [INFO] [stdout] | [INFO] [stdout] 27 | const ACC_STRICT: U2 = 0x0800; // Declared strictfp; floating-point mode is FPstrict. [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_SYNTHETIC` is never used [INFO] [stdout] --> src/bytecode/method.rs:29:7 [INFO] [stdout] | [INFO] [stdout] 29 | const ACC_SYNTHETIC: U2 = 0x1000; // Declared synthetic; not present in the source code. [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `allocate_array_in_region` is never used [INFO] [stdout] --> src/mem/heap.rs:132:8 [INFO] [stdout] | [INFO] [stdout] 36 | impl Heap { [INFO] [stdout] | --------- associated function in this implementation [INFO] [stdout] ... [INFO] [stdout] 132 | fn allocate_array_in_region( [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `max_stack_size` is never read [INFO] [stdout] --> src/mem/stack.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct JavaStack { [INFO] [stdout] | --------- field in this struct [INFO] [stdout] ... [INFO] [stdout] 12 | max_stack_size: usize, [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `is_subclass` is never used [INFO] [stdout] --> src/mem/stack.rs:359:4 [INFO] [stdout] | [INFO] [stdout] 359 | fn is_subclass(klass: &Klass, target: &str) -> bool { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:52:28 [INFO] [stdout] | [INFO] [stdout] 52 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 52 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 52 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:58:28 [INFO] [stdout] | [INFO] [stdout] 58 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 58 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 58 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:92:28 [INFO] [stdout] | [INFO] [stdout] 92 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 92 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 92 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:97:12 [INFO] [stdout] | [INFO] [stdout] 97 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 97 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 97 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/atom.rs:26:18 [INFO] [stdout] | [INFO] [stdout] 26 | unsafe { transmute::<[u8; 2], u16>(u) } [INFO] [stdout] | -------------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u16::from_ne_bytes` [INFO] [stdout] | [INFO] [stdout] = help: there's also `from_le_bytes` and `from_be_bytes` if you expect a particular byte order [INFO] [stdout] = note: `#[warn(unnecessary_transmutes)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/atom.rs:40:18 [INFO] [stdout] | [INFO] [stdout] 40 | unsafe { transmute::<[u8; 4], u32>(u) } [INFO] [stdout] | -------------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u32::from_ne_bytes` [INFO] [stdout] | [INFO] [stdout] = help: there's also `from_le_bytes` and `from_be_bytes` if you expect a particular byte order [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/atom.rs:58:18 [INFO] [stdout] | [INFO] [stdout] 58 | unsafe { transmute::<[u8; 8], u64>(u) } [INFO] [stdout] | -------------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u64::from_ne_bytes` [INFO] [stdout] | [INFO] [stdout] = help: there's also `from_le_bytes` and `from_be_bytes` if you expect a particular byte order [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:164:43 [INFO] [stdout] | [INFO] [stdout] 164 | let i: i32 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u32::cast_signed` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:169:43 [INFO] [stdout] | [INFO] [stdout] 169 | let i: f32 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `f32::from_bits` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:174:43 [INFO] [stdout] | [INFO] [stdout] 174 | let i: i64 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u64::cast_signed` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:181:43 [INFO] [stdout] | [INFO] [stdout] 181 | let i: f64 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `f64::from_bits` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/mod.rs:73:17 [INFO] [stdout] | [INFO] [stdout] 73 | &vv[..].copy_from_slice(&v); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 73 | let _ = &vv[..].copy_from_slice(&v); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/mod.rs:79:17 [INFO] [stdout] | [INFO] [stdout] 79 | &vv[..].copy_from_slice(&v); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 79 | let _ = &vv[..].copy_from_slice(&v); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/mod.rs:84:17 [INFO] [stdout] | [INFO] [stdout] 84 | &vv[..].copy_from_slice(&v); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 84 | let _ = &vv[..].copy_from_slice(&v); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/mem/heap.rs:50:13 [INFO] [stdout] | [INFO] [stdout] 50 | / HEAP.replace(Heap { [INFO] [stdout] 51 | | _data: data, [INFO] [stdout] 52 | | base: ptr, [INFO] [stdout] 53 | | from: Arc::new(RwLock::new(from)), [INFO] [stdout] ... | [INFO] [stdout] 60 | | old_size: old_size, [INFO] [stdout] 61 | | }); [INFO] [stdout] | |______________^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] = note: `#[warn(static_mut_refs)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: a dangling pointer will be produced because the temporary `[u8; 24]` will be dropped [INFO] [stdout] --> src/mem/heap.rs:124:48 [INFO] [stdout] | [INFO] [stdout] 124 | let obj_ptr = obj_header.into_vm_raw().as_ptr(); [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will immediately be invalid [INFO] [stdout] | | [INFO] [stdout] | this `[u8; 24]` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `[u8; 24]` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: you must make sure that the variable you bind the `[u8; 24]` to lives at least as long as the pointer returned by the call to `as_ptr` [INFO] [stdout] = help: in particular, if this pointer is returned from the current function, binding the `[u8; 24]` inside the function will not suffice [INFO] [stdout] = help: for more information, see [INFO] [stdout] = note: `#[warn(dangling_pointers_from_temporaries)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: a dangling pointer will be produced because the temporary `[u8; 24]` will be dropped [INFO] [stdout] --> src/mem/heap.rs:143:52 [INFO] [stdout] | [INFO] [stdout] 143 | let array_ptr = array_header.into_vm_raw().as_ptr(); [INFO] [stdout] | -------------------------- ^^^^^^ this pointer will immediately be invalid [INFO] [stdout] | | [INFO] [stdout] | this `[u8; 24]` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `[u8; 24]` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: you must make sure that the variable you bind the `[u8; 24]` to lives at least as long as the pointer returned by the call to `as_ptr` [INFO] [stdout] = help: in particular, if this pointer is returned from the current function, binding the `[u8; 24]` inside the function will not suffice [INFO] [stdout] = help: for more information, see [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: a dangling pointer will be produced because the temporary `[u8; 24]` will be dropped [INFO] [stdout] --> src/mem/heap.rs:161:63 [INFO] [stdout] | [INFO] [stdout] 161 | let array_header_ptr = array_header.into_vm_raw().as_ptr(); [INFO] [stdout] | -------------------------- ^^^^^^ this pointer will immediately be invalid [INFO] [stdout] | | [INFO] [stdout] | this `[u8; 24]` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `[u8; 24]` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: you must make sure that the variable you bind the `[u8; 24]` to lives at least as long as the pointer returned by the call to `as_ptr` [INFO] [stdout] = help: in particular, if this pointer is returned from the current function, binding the `[u8; 24]` inside the function will not suffice [INFO] [stdout] = help: for more information, see [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/mem/metaspace.rs:59:18 [INFO] [stdout] | [INFO] [stdout] 59 | unsafe { CLASSES.replace(Arc::new(arena)) }; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `init_value` is never used [INFO] [stdout] --> src/bytecode/field.rs:93:4 [INFO] [stdout] | [INFO] [stdout] 93 | fn init_value(access_flag: u16, descriptor: &str) -> Option { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_PRIVATE` is never used [INFO] [stdout] --> src/bytecode/method.rs:9:7 [INFO] [stdout] | [INFO] [stdout] 9 | const ACC_PRIVATE: U2 = 0x0002; // Declared private; accessible only within the defining class. [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_BRIDGE` is never used [INFO] [stdout] --> src/bytecode/method.rs:19:7 [INFO] [stdout] | [INFO] [stdout] 19 | const ACC_BRIDGE: U2 = 0x0040; // A bridge method, generated by the compiler. [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_VARARGS` is never used [INFO] [stdout] --> src/bytecode/method.rs:21:7 [INFO] [stdout] | [INFO] [stdout] 21 | const ACC_VARARGS: U2 = 0x0080; // Declared with variable number of arguments. [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_ABSTRACT` is never used [INFO] [stdout] --> src/bytecode/method.rs:25:7 [INFO] [stdout] | [INFO] [stdout] 25 | const ACC_ABSTRACT: U2 = 0x0400; // Declared abstract; no implementation is provided. [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_STRICT` is never used [INFO] [stdout] --> src/bytecode/method.rs:27:7 [INFO] [stdout] | [INFO] [stdout] 27 | const ACC_STRICT: U2 = 0x0800; // Declared strictfp; floating-point mode is FPstrict. [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `ACC_SYNTHETIC` is never used [INFO] [stdout] --> src/bytecode/method.rs:29:7 [INFO] [stdout] | [INFO] [stdout] 29 | const ACC_SYNTHETIC: U2 = 0x1000; // Declared synthetic; not present in the source code. [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `allocate_array_in_region` is never used [INFO] [stdout] --> src/mem/heap.rs:132:8 [INFO] [stdout] | [INFO] [stdout] 36 | impl Heap { [INFO] [stdout] | --------- associated function in this implementation [INFO] [stdout] ... [INFO] [stdout] 132 | fn allocate_array_in_region( [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `max_stack_size` is never read [INFO] [stdout] --> src/mem/stack.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 9 | pub struct JavaStack { [INFO] [stdout] | --------- field in this struct [INFO] [stdout] ... [INFO] [stdout] 12 | max_stack_size: usize, [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `is_subclass` is never used [INFO] [stdout] --> src/mem/stack.rs:359:4 [INFO] [stdout] | [INFO] [stdout] 359 | fn is_subclass(klass: &Klass, target: &str) -> bool { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:52:28 [INFO] [stdout] | [INFO] [stdout] 52 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] = note: `#[warn(non_fmt_panics)]` on by default [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 52 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 52 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:58:28 [INFO] [stdout] | [INFO] [stdout] 58 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 58 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 58 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:92:28 [INFO] [stdout] | [INFO] [stdout] 92 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 92 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 92 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/bytecode/mod.rs:97:12 [INFO] [stdout] | [INFO] [stdout] 97 | panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 97 - panic!(format!("Illegal method descriptor: {}", descriptor)); [INFO] [stdout] 97 + panic!("Illegal method descriptor: {}", descriptor); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/atom.rs:26:18 [INFO] [stdout] | [INFO] [stdout] 26 | unsafe { transmute::<[u8; 2], u16>(u) } [INFO] [stdout] | -------------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u16::from_ne_bytes` [INFO] [stdout] | [INFO] [stdout] = help: there's also `from_le_bytes` and `from_be_bytes` if you expect a particular byte order [INFO] [stdout] = note: `#[warn(unnecessary_transmutes)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/atom.rs:40:18 [INFO] [stdout] | [INFO] [stdout] 40 | unsafe { transmute::<[u8; 4], u32>(u) } [INFO] [stdout] | -------------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u32::from_ne_bytes` [INFO] [stdout] | [INFO] [stdout] = help: there's also `from_le_bytes` and `from_be_bytes` if you expect a particular byte order [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/atom.rs:58:18 [INFO] [stdout] | [INFO] [stdout] 58 | unsafe { transmute::<[u8; 8], u64>(u) } [INFO] [stdout] | -------------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u64::from_ne_bytes` [INFO] [stdout] | [INFO] [stdout] = help: there's also `from_le_bytes` and `from_be_bytes` if you expect a particular byte order [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:164:43 [INFO] [stdout] | [INFO] [stdout] 164 | let i: i32 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u32::cast_signed` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:169:43 [INFO] [stdout] | [INFO] [stdout] 169 | let i: f32 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `f32::from_bits` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:174:43 [INFO] [stdout] | [INFO] [stdout] 174 | let i: i64 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u64::cast_signed` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/bytecode/constant_pool.rs:181:43 [INFO] [stdout] | [INFO] [stdout] 181 | let i: f64 = unsafe { transmute::(v) }; [INFO] [stdout] | ---------------------^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `f64::from_bits` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/mod.rs:73:17 [INFO] [stdout] | [INFO] [stdout] 73 | &vv[..].copy_from_slice(&v); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 73 | let _ = &vv[..].copy_from_slice(&v); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/mod.rs:79:17 [INFO] [stdout] | [INFO] [stdout] 79 | &vv[..].copy_from_slice(&v); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 79 | let _ = &vv[..].copy_from_slice(&v); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/mod.rs:84:17 [INFO] [stdout] | [INFO] [stdout] 84 | &vv[..].copy_from_slice(&v); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 84 | let _ = &vv[..].copy_from_slice(&v); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: non-binding let on a synchronization lock [INFO] [stdout] --> src/mem/metaspace.rs:86:21 [INFO] [stdout] | [INFO] [stdout] 86 | let _ = class_arena!().mutex.lock().unwrap(); [INFO] [stdout] | ^ this lock is not assigned to a binding and is immediately dropped [INFO] [stdout] | [INFO] [stdout] = note: `#[deny(let_underscore_lock)]` on by default [INFO] [stdout] help: consider binding to an unused variable to avoid immediately dropping the value [INFO] [stdout] | [INFO] [stdout] 86 | let _unused = class_arena!().mutex.lock().unwrap(); [INFO] [stdout] | ++++++ [INFO] [stdout] help: consider immediately dropping the value [INFO] [stdout] | [INFO] [stdout] 86 - let _ = class_arena!().mutex.lock().unwrap(); [INFO] [stdout] 86 + drop(class_arena!().mutex.lock().unwrap()); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/mem/heap.rs:50:13 [INFO] [stdout] | [INFO] [stdout] 50 | / HEAP.replace(Heap { [INFO] [stdout] 51 | | _data: data, [INFO] [stdout] 52 | | base: ptr, [INFO] [stdout] 53 | | from: Arc::new(RwLock::new(from)), [INFO] [stdout] ... | [INFO] [stdout] 60 | | old_size: old_size, [INFO] [stdout] 61 | | }); [INFO] [stdout] | |______________^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] = note: `#[warn(static_mut_refs)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/klass.rs:128:9 [INFO] [stdout] | [INFO] [stdout] 128 | &klass.build_vtable(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 128 | let _ = &klass.build_vtable(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: a dangling pointer will be produced because the temporary `[u8; 24]` will be dropped [INFO] [stdout] --> src/mem/heap.rs:124:48 [INFO] [stdout] | [INFO] [stdout] 124 | let obj_ptr = obj_header.into_vm_raw().as_ptr(); [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will immediately be invalid [INFO] [stdout] | | [INFO] [stdout] | this `[u8; 24]` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `[u8; 24]` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: you must make sure that the variable you bind the `[u8; 24]` to lives at least as long as the pointer returned by the call to `as_ptr` [INFO] [stdout] = help: in particular, if this pointer is returned from the current function, binding the `[u8; 24]` inside the function will not suffice [INFO] [stdout] = help: for more information, see [INFO] [stdout] = note: `#[warn(dangling_pointers_from_temporaries)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/klass.rs:129:9 [INFO] [stdout] | [INFO] [stdout] 129 | &klass.build_itable(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 129 | let _ = &klass.build_itable(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/klass.rs:130:9 [INFO] [stdout] | [INFO] [stdout] 130 | &klass.build_layout(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 130 | let _ = &klass.build_layout(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: a dangling pointer will be produced because the temporary `[u8; 24]` will be dropped [INFO] [stdout] --> src/mem/heap.rs:143:52 [INFO] [stdout] | [INFO] [stdout] 143 | let array_ptr = array_header.into_vm_raw().as_ptr(); [INFO] [stdout] | -------------------------- ^^^^^^ this pointer will immediately be invalid [INFO] [stdout] | | [INFO] [stdout] | this `[u8; 24]` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `[u8; 24]` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: you must make sure that the variable you bind the `[u8; 24]` to lives at least as long as the pointer returned by the call to `as_ptr` [INFO] [stdout] = help: in particular, if this pointer is returned from the current function, binding the `[u8; 24]` inside the function will not suffice [INFO] [stdout] = help: for more information, see [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: a dangling pointer will be produced because the temporary `[u8; 24]` will be dropped [INFO] [stdout] --> src/mem/heap.rs:161:63 [INFO] [stdout] | [INFO] [stdout] 161 | let array_header_ptr = array_header.into_vm_raw().as_ptr(); [INFO] [stdout] | -------------------------- ^^^^^^ this pointer will immediately be invalid [INFO] [stdout] | | [INFO] [stdout] | this `[u8; 24]` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `[u8; 24]` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: you must make sure that the variable you bind the `[u8; 24]` to lives at least as long as the pointer returned by the call to `as_ptr` [INFO] [stdout] = help: in particular, if this pointer is returned from the current function, binding the `[u8; 24]` inside the function will not suffice [INFO] [stdout] = help: for more information, see [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/mem/metaspace.rs:59:18 [INFO] [stdout] | [INFO] [stdout] 59 | unsafe { CLASSES.replace(Arc::new(arena)) }; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/mem/strings.rs:26:13 [INFO] [stdout] | [INFO] [stdout] 26 | STRINGS.replace(Strings(RwLock::new(HashMap::with_capacity(4096)))); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: non-binding let on a synchronization lock [INFO] [stdout] --> src/mem/metaspace.rs:86:21 [INFO] [stdout] | [INFO] [stdout] 86 | let _ = class_arena!().mutex.lock().unwrap(); [INFO] [stdout] | ^ this lock is not assigned to a binding and is immediately dropped [INFO] [stdout] | [INFO] [stdout] = note: `#[deny(let_underscore_lock)]` on by default [INFO] [stdout] help: consider binding to an unused variable to avoid immediately dropping the value [INFO] [stdout] | [INFO] [stdout] 86 | let _unused = class_arena!().mutex.lock().unwrap(); [INFO] [stdout] | ++++++ [INFO] [stdout] help: consider immediately dropping the value [INFO] [stdout] | [INFO] [stdout] 86 - let _ = class_arena!().mutex.lock().unwrap(); [INFO] [stdout] 86 + drop(class_arena!().mutex.lock().unwrap()); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/klass.rs:128:9 [INFO] [stdout] | [INFO] [stdout] 128 | &klass.build_vtable(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 128 | let _ = &klass.build_vtable(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/klass.rs:129:9 [INFO] [stdout] | [INFO] [stdout] 129 | &klass.build_itable(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 129 | let _ = &klass.build_itable(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/interpreter/mod.rs:504:21 [INFO] [stdout] | [INFO] [stdout] 504 | / &field.value.set(match t.as_ref() { [INFO] [stdout] 505 | | "D" | "J" => Some(Value::eval_w(context.stack.pop_w())), [INFO] [stdout] 506 | | _ => Some(Value::eval(context.stack.pop(), &t)), [INFO] [stdout] 507 | | }); [INFO] [stdout] | |______________________^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 504 | let _ = &field.value.set(match t.as_ref() { [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/mem/klass.rs:130:9 [INFO] [stdout] | [INFO] [stdout] 130 | &klass.build_layout(); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 130 | let _ = &klass.build_layout(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/mem/strings.rs:26:13 [INFO] [stdout] | [INFO] [stdout] 26 | STRINGS.replace(Strings(RwLock::new(HashMap::with_capacity(4096)))); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused borrow that must be used [INFO] [stdout] --> src/interpreter/mod.rs:504:21 [INFO] [stdout] | [INFO] [stdout] 504 | / &field.value.set(match t.as_ref() { [INFO] [stdout] 505 | | "D" | "J" => Some(Value::eval_w(context.stack.pop_w())), [INFO] [stdout] 506 | | _ => Some(Value::eval(context.stack.pop(), &t)), [INFO] [stdout] 507 | | }); [INFO] [stdout] | |______________________^ the borrow produces a value [INFO] [stdout] | [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 504 | let _ = &field.value.set(match t.as_ref() { [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/interpreter/mod.rs:707:25 [INFO] [stdout] | [INFO] [stdout] 707 | _ => panic!(format!( [INFO] [stdout] | _________________________^ [INFO] [stdout] 708 | | "Instruction 0x{:2x?} not implemented yet.", [INFO] [stdout] 709 | | instruction [INFO] [stdout] 710 | | )), [INFO] [stdout] | |_____________^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 707 ~ _ => panic!( [INFO] [stdout] 708 | "Instruction 0x{:2x?} not implemented yet.", [INFO] [stdout] 709 | instruction [INFO] [stdout] 710 ~ ), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/interpreter/thread.rs:36:13 [INFO] [stdout] | [INFO] [stdout] 36 | / THREADS.replace(ThreadGroup { [INFO] [stdout] 37 | | threads: Arc::new(Mutex::new(BTreeMap::new())), [INFO] [stdout] 38 | | }); [INFO] [stdout] | |______________^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/interpreter/thread.rs:56:37 [INFO] [stdout] | [INFO] [stdout] 56 | Err(no_class) => panic!(format!("ClassNotFoundException: {}", no_class)), [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 56 - Err(no_class) => panic!(format!("ClassNotFoundException: {}", no_class)), [INFO] [stdout] 56 + Err(no_class) => panic!("ClassNotFoundException: {}", no_class), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/gc/mod.rs:33:9 [INFO] [stdout] | [INFO] [stdout] 33 | GC_TX.replace(tx); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/interpreter/mod.rs:707:25 [INFO] [stdout] | [INFO] [stdout] 707 | _ => panic!(format!( [INFO] [stdout] | _________________________^ [INFO] [stdout] 708 | | "Instruction 0x{:2x?} not implemented yet.", [INFO] [stdout] 709 | | instruction [INFO] [stdout] 710 | | )), [INFO] [stdout] | |_____________^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 707 ~ _ => panic!( [INFO] [stdout] 708 | "Instruction 0x{:2x?} not implemented yet.", [INFO] [stdout] 709 | instruction [INFO] [stdout] 710 ~ ), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/interpreter/mod.rs:865:24 [INFO] [stdout] | [INFO] [stdout] 865 | let sfr = unsafe { std::mem::transmute::(std::mem::transmute::(nu) >> 1) }; [INFO] [stdout] | -------------------------------^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `u32::cast_signed` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary transmute [INFO] [stdout] --> src/interpreter/mod.rs:865:56 [INFO] [stdout] | [INFO] [stdout] 865 | let sfr = unsafe { std::mem::transmute::(std::mem::transmute::(nu) >> 1) }; [INFO] [stdout] | -------------------------------^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: replace this with: `i32::cast_unsigned` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/interpreter/thread.rs:36:13 [INFO] [stdout] | [INFO] [stdout] 36 | / THREADS.replace(ThreadGroup { [INFO] [stdout] 37 | | threads: Arc::new(Mutex::new(BTreeMap::new())), [INFO] [stdout] 38 | | }); [INFO] [stdout] | |______________^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: panic message is not a string literal [INFO] [stdout] --> src/interpreter/thread.rs:56:37 [INFO] [stdout] | [INFO] [stdout] 56 | Err(no_class) => panic!(format!("ClassNotFoundException: {}", no_class)), [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021 [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here [INFO] [stdout] help: remove the `format!(..)` macro call [INFO] [stdout] | [INFO] [stdout] 56 - Err(no_class) => panic!(format!("ClassNotFoundException: {}", no_class)), [INFO] [stdout] 56 + Err(no_class) => panic!("ClassNotFoundException: {}", no_class), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: creating a mutable reference to mutable static [INFO] [stdout] --> src/gc/mod.rs:33:9 [INFO] [stdout] | [INFO] [stdout] 33 | GC_TX.replace(tx); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ mutable reference to mutable static [INFO] [stdout] | [INFO] [stdout] = note: for more information, see [INFO] [stdout] = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `azeroth-vm` (lib) due to 1 previous error; 45 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] error: could not compile `azeroth-vm` (lib test) due to 1 previous error; 48 warnings emitted [INFO] running `Command { std: "docker" "inspect" "5ef55410bb0c377665a50ac56ec13817ffee1148611dbd2f7626743706b7dfb0", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "5ef55410bb0c377665a50ac56ec13817ffee1148611dbd2f7626743706b7dfb0", kill_on_drop: false }` [INFO] [stdout] 5ef55410bb0c377665a50ac56ec13817ffee1148611dbd2f7626743706b7dfb0