rust/clippy: fix lint: while_let_loop

diff --git a/rust/src/lib.rs b/rust/src/lib.rs
index 346bbe8cfed57b004dc2415807f1f6ad70a73a54..a0f00facc0bd0e9f204a81c54d159dcef64746f3 100644 (file)
--- a/rust/src/lib.rs
+++ b/rust/src/lib.rs
@@ -50,7 +50,6 @@
 #![allow(clippy::single_match)]
 #![allow(clippy::type_complexity)]
 #![allow(clippy::upper_case_acronyms)]
-#![allow(clippy::while_let_loop)]
 
 #[macro_use]
 extern crate bitflags;

diff --git a/rust/src/rdp/util.rs b/rust/src/rdp/util.rs
index d58ac94595a3bd73992814805bf1e34131e6947b..a4228f20373b23df9c8495f2bc5f1eac6a43475f 100644 (file)
--- a/rust/src/rdp/util.rs
+++ b/rust/src/rdp/util.rs
@@ -28,16 +28,11 @@ use widestring::U16CString;
 pub fn le_slice_to_string(input: &[u8]) -> Result<String, Box<dyn std::error::Error>> {
     let mut vec = Vec::new();
     let mut cursor = Cursor::new(input);
-    loop {
-        match cursor.read_u16::<byteorder::LittleEndian>() {
-            Ok(x) => {
-                if x == 0 {
-                    break;
-                };
-                vec.push(x)
-            }
-            Err(_) => break,
+    while let Ok(x) = cursor.read_u16::<byteorder::LittleEndian>() {
+        if x == 0 {
+            break;
         }
+        vec.push(x);
     }
     match U16CString::new(vec) {
         Ok(x) => match x.to_string() {