Implement PartialOrd and Ord for http::Uri

I thought about an implementation like this

impl PartialOrd for Uri {
    fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
        Some(self.cmp(other))
    }
}

impl Ord for Uri {
    fn cmp(&self, other: &Self) -> Ordering {
        if self.eq(other) {
            Ordering::Equal
        } else {
            (self.scheme(), self.authority(), self.path_and_query())
                .cmp(&(other.scheme(), other.authority(), other.path_and_query()))
        }
    }
}

the problem is the components only implement PartialOrd.

Another option would be:

impl PartialOrd for Uri {
    fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
        Some(self.cmp(other))
    }
}

impl Ord for Uri {
    fn cmp(&self, other: &Self) -> Ordering {
        if self.eq(other) {
            Ordering::Equal
        } else {
            self.to_string().cmp(other.to_string())
        }
    }
}

Start by reviewing http::Uri's existing equality and component comparison behavior, then resolve whether ordering should use its components or string form. Done means PartialOrd and Ord provide a consistent ordering with coverage for the relevant URI cases.