Move length overflow check to parser.rs and clarify doc

src/internal.rs

@@ -19,21 +19,12 @@ pub trait ToUri {
     fn to_uri(self) -> Result<Uri<Self::Val>, Self::Err>;
 }
 
-#[cold]
-fn len_overflow() -> ! {
-    panic!("input length > u32::MAX");
-}
-
 impl<'a> ToUri for &'a str {
     type Val = &'a str;
     type Err = ParseError;
 
     #[inline]
     fn to_uri(self) -> Result<Uri<Self::Val>, Self::Err> {
-        if self.len() > u32::MAX as usize {
-            len_overflow();
-        }
-
         let meta = parser::parse(self.as_bytes())?;
         Ok(Uri { val: self, meta })
     }
@@ -45,10 +36,6 @@ impl ToUri for String {
 
     #[inline]
     fn to_uri(self) -> Result<Uri<Self::Val>, Self::Err> {
-        if self.len() > u32::MAX as usize {
-            len_overflow();
-        }
-
         match parser::parse(self.as_bytes()) {
             Ok(meta) => Ok(Uri { val: self, meta }),
             Err(e) => Err(e.with_input(self)),

src/lib.rs

@@ -130,8 +130,8 @@ impl<T> Uri<T> {
     /// - `Result<Uri<&str>, ParseError>` for `I = &str`;
     /// - `Result<Uri<String>, ParseError<String>>` for `I = String`.
     ///
-    /// Returns `Ok` if and only if the string matches the [`URI-reference`]
-    /// ABNF rule from RFC 3986.
+    /// When not panicking, returns `Ok` if and only if the string matches
+    /// the [`URI-reference`] ABNF rule from RFC 3986.
     ///
     /// You may recover an input [`String`] by calling [`ParseError::into_input`].
     ///

src/parser.rs

@@ -12,6 +12,10 @@ use core::{
 type Result<T> = core::result::Result<T, ParseError>;
 
 pub(crate) fn parse(bytes: &[u8]) -> Result<Meta> {
+    if bytes.len() > u32::MAX as usize {
+        panic!("input length > u32::MAX");
+    }
+
     let mut parser = Parser {
         reader: Reader::new(bytes),
         out: Meta::default(),