line 1352, change self to (*self), other to (*other)

The current code will not results bug, but it difficult to understand. These code result to call &f32::partial_cmp(), and the performance will be lower than the changed code. I'm not sure why the current code don't use (*self) (*other), if you have some idea, please let me know.
This commit is contained in:
Warrenren 2022-06-10 19:08:03 +08:00 committed by GitHub
parent 3dea0033f7
commit 5e9e73cc9f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1349,7 +1349,7 @@ mod impls {
impl PartialOrd for $t {
#[inline]
fn partial_cmp(&self, other: &$t) -> Option<Ordering> {
match (self <= other, self >= other) {
match ((*self) <= (*other), (*self) >= (*other)) {
(false, false) => None,
(false, true) => Some(Greater),
(true, false) => Some(Less),