Use .to_string() instead of String::from_utf8_lossy

Ralith · May 19, 2024 · 547deb1 · 547deb1
1 parent 8f5b67e
commit 547deb1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/generator/src/main.rs b/generator/src/main.rs
@@ -1520,7 +1520,10 @@ impl Parser {
                             bytes => {
                                 let cstr = CStr::from_bytes_with_nul(bytes)
                                     .expect("extension names should be null terminated strings");
-                                let string = String::from_utf8_lossy(cstr.to_bytes()).to_string();
+                                let string = cstr
+                                    .to_str()
+                                    .expect("extension names should be valid UTF-8")
+                                    .to_string();
                                 out.other.push(string);
                             }
                         }

diff --git a/openxr/src/generated.rs b/openxr/src/generated.rs
@@ -689,7 +689,10 @@ impl ExtensionSet {
                 bytes => {
                     let cstr = CStr::from_bytes_with_nul(bytes)
                         .expect("extension names should be null terminated strings");
-                    let string = String::from_utf8_lossy(cstr.to_bytes()).to_string();
+                    let string = cstr
+                        .to_str()
+                        .expect("extension names should be valid UTF-8")
+                        .to_string();
                     out.other.push(string);
                 }
             }