feat: Implement TryFrom<GitUrl> for url::Url (#66)

`GitUrl::url_compat_display()` is currently private, but it would be
very useful to have this functionality - in my case, normalizing an SSH
style URL is fine, especially because I do not want to bother with
handling the `'url` lifetime of `GitUrl` in my wrapper enum.

This change implements fallible conversion to `url::Url` for `GitUrl`
and `&GitUrl` via `GitUrl::url_compat_display()`.

Longer term it might be wise to re-visit the use of a lifetime for the
purpose of re-using slices of the input string - the `url::Url` crate
does not bother with such micro-optimizations either, presumably for
good reasons.

Obviously, to use a `GitUrl` as a `url::Url`, the conversion to
`url::Url` still has to happen twice (because `GitUrl::parse()` incurs
the conversion via `GitUrl::is_valid()`), but that can be optimized in a
separate change, e.g. via a new `GitUrl::parse_to_url()`, or by removing
the conversion from `GitUrl::is_valid()` altogether, leaving it to the
caller to attempt the conversion.

Author: dzuelke

src/types/mod.rs

@@ -12,6 +12,7 @@ pub use error::GitUrlParseError;
 
 use core::str;
 use std::fmt;
+use url::Url;
 
 use getset::{CloneGetters, CopyGetters, Setters};
 #[cfg(feature = "log")]
@@ -140,6 +141,24 @@ impl<'url> GitUrl<'url> {
     }
 }
 
+#[cfg(feature = "url")]
+impl<'url> TryFrom<&GitUrl<'url>> for Url {
+    type Error = url::ParseError;
+    fn try_from(value: &GitUrl) -> Result<Self, Self::Error> {
+        // Since we don't fully implement any spec, we'll rely on the url crate
+        Url::parse(&value.url_compat_display())
+    }
+}
+
+#[cfg(feature = "url")]
+impl<'url> TryFrom<GitUrl<'url>> for Url {
+    type Error = url::ParseError;
+    fn try_from(value: GitUrl) -> Result<Self, Self::Error> {
+        // Since we don't fully implement any spec, we'll rely on the url crate
+        Url::parse(&value.url_compat_display())
+    }
+}
+
 impl<'url> GitUrl<'url> {
     /// Returns `GitUrl` after removing all user info values
     pub fn trim_auth(&self) -> GitUrl {
@@ -325,7 +344,7 @@ impl<'url> GitUrl<'url> {
         #[cfg(feature = "url")]
         {
             // Since we don't fully implement any spec, we'll rely on the url crate
-            let _u = url::Url::parse(&self.url_compat_display())?;
+            let _u: Url = self.try_into()?;
         }
 
         Ok(())