refactor(embedded): Pull out sanitization logic

Author: epage

src/cargo/util/toml/embedded.rs

@@ -183,25 +183,7 @@ impl RawScript {
             .file_stem()
             .ok_or_else(|| anyhow::format_err!("no file name"))?
             .to_string_lossy();
-        let mut slug = String::new();
-        for (i, c) in name.chars().enumerate() {
-            match (i, c) {
-                (0, '0'..='9') => {
-                    slug.push('_');
-                    slug.push(c);
-                }
-                (_, '0'..='9') | (_, 'a'..='z') | (_, '_') | (_, '-') => {
-                    slug.push(c);
-                }
-                (_, 'A'..='Z') => {
-                    // Convert uppercase characters to lowercase to avoid `non_snake_case` warnings.
-                    slug.push(c.to_ascii_lowercase());
-                }
-                (_, _) => {
-                    slug.push('_');
-                }
-            }
-        }
+        let slug = sanitize_package_name(&name);
         Ok(slug)
     }
 
@@ -210,6 +192,29 @@ impl RawScript {
     }
 }
 
+fn sanitize_package_name(name: &str) -> String {
+    let mut slug = String::new();
+    for (i, c) in name.chars().enumerate() {
+        match (i, c) {
+            (0, '0'..='9') => {
+                slug.push('_');
+                slug.push(c);
+            }
+            (_, '0'..='9') | (_, 'a'..='z') | (_, '_') | (_, '-') => {
+                slug.push(c);
+            }
+            (_, 'A'..='Z') => {
+                // Convert uppercase characters to lowercase to avoid `non_snake_case` warnings.
+                slug.push(c.to_ascii_lowercase());
+            }
+            (_, _) => {
+                slug.push('_');
+            }
+        }
+    }
+    slug
+}
+
 fn default_target_dir() -> CargoResult<std::path::PathBuf> {
     let mut cargo_home = home::cargo_home()?;
     cargo_home.push("eval");