Update to Rust 1.59.0 (#718)

* Update to Rust 1.59.0 * Fix clippy::single_char_pattern in console, ui/logo * Fix clippy::needless_late_init in load_block_entities
iceiix · Aug 28, 2022 · bbbe58a · bbbe58a
1 parent 303783f
commit bbbe58a
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 12 deletions.
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -11,7 +11,7 @@ jobs:
         uses: actions-rs/toolchain@v1
         with:
           # TODO: refactor toolchain version
-          toolchain: 1.58.1
+          toolchain: 1.59.0
           components: clippy, rustfmt
           default: true
       - name: Install dependencies
@@ -60,7 +60,7 @@ jobs:
       - name: Install Rust
         uses: actions-rs/toolchain@v1
         with:
-          toolchain: 1.58.1
+          toolchain: 1.59.0
           components: clippy, rustfmt
           default: true
       - name: Build binary
@@ -105,7 +105,7 @@ jobs:
       - name: Install Rust
         uses: actions-rs/toolchain@v1
         with:
-          toolchain: 1.58.1
+          toolchain: 1.59.0
           components: clippy, rustfmt
           default: true
       - name: Build binary
@@ -161,7 +161,7 @@ jobs:
         uses: actions-rs/toolchain@v1
         with:
           # TODO: refactor toolchain version
-          toolchain: 1.58.1
+          toolchain: 1.59.0
           components: clippy, rustfmt
           default: true
       - name: Install wasm-pack

diff --git a/Cargo.toml b/Cargo.toml
@@ -3,7 +3,7 @@ name = "stevenarella"
 version = "0.0.1"
 authors = [ "Thinkofdeath <[email protected]>", "iceiix <[email protected]>" ]
 edition = "2021"
-rust-version = "1.58.1"
+rust-version = "1.59.0"
 resolver = "2"
 description = "Multi-protocol multi-platform Minecraft-compatible client"
 repository = "https://github.com/iceiix/stevenarella"

diff --git a/README.md b/README.md
@@ -70,7 +70,7 @@ from [GitHub Actions](https://actions-badge.atrox.dev/iceiix/stevenarella/goto?r
 
 ## Dependencies
 
-Requires Rust stable version 1.58.1 or newer.
+Requires Rust stable version 1.59.0 or newer.
 
 **Debian/Ubuntu**
 

diff --git a/src/console/mod.rs b/src/console/mod.rs
@@ -370,7 +370,7 @@ impl Console {
             }
         }
 
-        let mut file = &record.file().unwrap_or("").replace("\\", "/")[..];
+        let mut file = &record.file().unwrap_or("").replace('\\', "/")[..];
         if let Some(pos) = file.rfind("src/") {
             file = &file[pos + 4..];
         }

diff --git a/src/server/mod.rs b/src/server/mod.rs
@@ -1947,13 +1947,12 @@ impl Server {
             let z = block_entity.1.get("z").unwrap().as_int().unwrap();
             if let Some(tile_id) = block_entity.1.get("id") {
                 let tile_id = tile_id.as_str().unwrap();
-                let action;
-                match tile_id {
+                let action = match tile_id {
                     // Fake a sign update
-                    "Sign" => action = 9,
+                    "Sign" => 9,
                     // Not something we care about, so break the loop
                     _ => continue,
-                }
+                };
                 self.on_block_entity_update_u8(packet::play::clientbound::UpdateBlockEntity_u8 {
                     location: Position::new(x, y, z),
                     action,

diff --git a/src/ui/logo.rs b/src/ui/logo.rs
@@ -99,7 +99,7 @@ impl Logo {
                 let mut texts = String::new();
                 file.read_to_string(&mut texts).unwrap();
                 for line in texts.lines() {
-                    text_strings.push(line.to_owned().replace("\r", ""));
+                    text_strings.push(line.to_owned().replace('\r', ""));
                 }
             }
             let mut r: rand_pcg::Pcg32 =