more fixes

Signed-off-by: Adam Gutglick <adam@spiraldb.com>

Author: AdamGS

vortex-array/src/expr/stats/bound.rs

@@ -248,7 +248,7 @@ impl<T: PartialOrd> PartialOrd<T> for UpperBound<T> {
                     Some(o)
                 }
             }),
-            Absent => return None,
+            Absent => None,
         }
     }
 }

vortex-array/src/expr/stats/precision.rs

@@ -22,19 +22,14 @@ use crate::scalar::ScalarValue;
 /// This is statistic specific, for max this will be an upper bound. Meaning that the actual max
 /// in an array is guaranteed to be less than or equal to the inexact value, but equal to the exact
 /// value.
-#[derive(Debug, PartialEq, Eq, Clone, Copy)]
+#[derive(Default, Debug, PartialEq, Eq, Clone, Copy)]
 pub enum Precision<T> {
     Exact(T),
     Inexact(T),
+    #[default]
     Absent,
 }
 
-impl<T> Default for Precision<T> {
-    fn default() -> Self {
-        Self::Absent
-    }
-}
-
 impl<T> Precision<Option<T>> {
     /// Transpose the `Precision<Option<T>>` into `Option<Precision<T>>`.
     pub fn transpose(self) -> Option<Precision<T>> {

vortex-cuda/src/layout.rs

@@ -434,8 +434,8 @@ fn truncate_scalar_stat<F: Fn(Scalar) -> Option<(Scalar, bool)>>(
     stat: Stat,
     truncation: F,
 ) {
-    if let Some(sv) = statistics.get(stat) {
-        if let Some((truncated_value, truncated)) = truncation(sv.into_inner()) {
+    if let Some(sv) = statistics.get(stat).into_inner() {
+        if let Some((truncated_value, truncated)) = truncation(sv) {
             if truncated && let Some(v) = truncated_value.into_value() {
                 statistics.set(stat, Precision::Inexact(v));
             }