Use split_once to simplify the code.

Co-authored-by: Jacob Pratt <jacob@jhpratt.dev>
rust-lang · Dec 30, 2021 · 2b4430c · 2b4430c
1 parent 8daf8e9
commit 2b4430c
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/models/krate.rs b/src/models/krate.rs
@@ -297,10 +297,8 @@ impl Crate {
 
     /// Validates a whole feature string, `features = ["THIS", "ALL/THIS"]`.
     pub fn valid_feature(name: &str) -> bool {
-        match name.find('/') {
-            Some(pos) => {
-                let (dep, dep_feat) = name.split_at(pos);
-                let dep_feat = &dep_feat[1..];
+        match name.split_once('/') {
+            Some((dep, dep_feat)) => {
                 let dep = dep.strip_suffix('?').unwrap_or(dep);
                 Crate::valid_feature_prefix(dep) && Crate::valid_feature_name(dep_feat)
             }