Remove ascii_part()

Name didn't fit in with the other methods, and can be replaced with `s.into_ascii().map_err(|err| s[..err.index()].into_ascii().unwrap() )`.

Author: tormol

src/ascii_str.rs

@@ -376,17 +376,13 @@ impl Error for IntoAsciiStrError {
 pub trait IntoAsciiStr : AsciiExt {
     /// Convert to `AsciiStr`, not doing any range asserts.
     unsafe fn into_ascii_unchecked(&self) -> &AsciiStr;
-    /// Convert the portion from start of self that is valid ASCII to `AsciiStr`.
-    fn ascii_part(&self) -> &AsciiStr;
     /// Convert to `AsciiStr`.
     fn into_ascii(&self) -> Result<&AsciiStr,IntoAsciiStrError>;
 }
 /// Trait for converting various slices into `AsciiStr`.
 pub trait IntoMutAsciiStr : AsciiExt {
     /// Convert to `AsciiStr`, not doing any range asserts.
     unsafe fn into_ascii_mut_unchecked(&mut self) -> &mut AsciiStr;
-    /// Convert the portion from start of self that is valid ASCII to `AsciiStr`.
-    fn ascii_part_mut(&mut self) -> &mut AsciiStr;
     /// Convert to `AsciiStr`.
     fn into_ascii_mut(&mut self) -> Result<&mut AsciiStr,IntoAsciiStrError>;
 }
@@ -396,9 +392,6 @@ impl IntoAsciiStr for AsciiStr {
     fn into_ascii(&self) -> Result<&AsciiStr,IntoAsciiStrError> {
         Ok(self)
     }
-    fn ascii_part(&self) -> &AsciiStr {
-        self
-    }
     unsafe fn into_ascii_unchecked(&self) -> &AsciiStr {
         self
     }
@@ -408,9 +401,6 @@ impl IntoMutAsciiStr for AsciiStr {
     fn into_ascii_mut(&mut self) -> Result<&mut AsciiStr,IntoAsciiStrError> {
         Ok(self)
     }
-    fn ascii_part_mut(&mut self) -> &mut AsciiStr {
-        self
-    }
     unsafe fn into_ascii_mut_unchecked(&mut self) -> &mut AsciiStr {
         self
     }
@@ -426,12 +416,6 @@ impl IntoAsciiStr for [u8] {
             None => unsafe{ Ok(self.into_ascii_unchecked()) },
         }
     }
-    fn ascii_part(&self) -> &AsciiStr {
-        let ascii = self.iter().position(|b| *b > 127 )
-                               .map(|l| &self[..l] )
-                               .unwrap_or(self);
-        unsafe{ ascii.into_ascii_unchecked() }
-    }
     unsafe fn into_ascii_unchecked(&self) -> &AsciiStr {
         AsciiStr::from_bytes_unchecked(self)
     }
@@ -446,32 +430,17 @@ impl IntoMutAsciiStr for [u8] {
             None => unsafe{ Ok(self.into_ascii_mut_unchecked()) },
         }
     }
-    fn ascii_part_mut(&mut self) -> &mut AsciiStr {
-        let ascii = match self.iter_mut().position(|b| *b > 127 ) {
-            Some(l) => &mut self[..l],
-            None    => self,
-        };
-        unsafe{ ascii.into_ascii_mut_unchecked() }
-    }
     unsafe fn into_ascii_mut_unchecked(&mut self) -> &mut AsciiStr {
         mem::transmute(self)
     }
 }
 
 impl IntoAsciiStr for str {
     fn into_ascii(&self) -> Result<&AsciiStr,IntoAsciiStrError> {
-        let s = self.ascii_part();
-        if s.len() == self.len() {
-            Ok(s)
-        } else {
-            Err(IntoAsciiStrError{
-                index: s.len(),
-                not_ascii: self[s.len()..].chars().next().unwrap(),
-            })
-        }
-    }
-    fn ascii_part(&self) -> &AsciiStr {
-        self.as_bytes().ascii_part()
+        self.as_bytes().into_ascii().map_err(|err| IntoAsciiStrError{
+            not_ascii: self[err.index..].chars().next().unwrap(),
+            index: err.index,
+        })
     }
     unsafe fn into_ascii_unchecked(&self) -> &AsciiStr {
         mem::transmute(self)
@@ -487,12 +456,6 @@ impl IntoMutAsciiStr for str {
             None => unsafe{ Ok(self.into_ascii_mut_unchecked()) },
         }
     }
-    fn ascii_part_mut(&mut self) -> &mut AsciiStr {
-        unsafe{ match self.bytes().position(|b| b > 127 ) {
-            Some(index) => self.slice_mut_unchecked(0, index).into_ascii_mut_unchecked(),
-            None => self.into_ascii_mut_unchecked(),
-        }}
-    }
     unsafe fn into_ascii_mut_unchecked(&mut self) -> &mut AsciiStr {
         mem::transmute(self)
     }