Make NaiveDateTime::and_utc const

chronotope · Sep 13, 2023 · 5f50ab5 · 5f50ab5
1 parent f542e57
commit 5f50ab5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/datetime/mod.rs b/src/datetime/mod.rs
@@ -617,6 +617,14 @@ impl DateTime<Utc> {
     pub fn from_timestamp(secs: i64, nsecs: u32) -> Option<Self> {
         NaiveDateTime::from_timestamp_opt(secs, nsecs).as_ref().map(NaiveDateTime::and_utc)
     }
+
+    // FIXME: remove when our MSRV is 1.61+
+    // This method is used by `NaiveDateTime::and_utc` because `DateTime::from_naive_utc_and_offset`
+    // can't be made const yet.
+    // Trait bounds in const function / implementation blocks were not supported until 1.61.
+    pub(crate) const fn from_naive_utc(datetime: NaiveDateTime) -> Self {
+        DateTime { datetime, offset: Utc }
+    }
 }
 
 impl Default for DateTime<Utc> {

diff --git a/src/naive/datetime/mod.rs b/src/naive/datetime/mod.rs
@@ -963,8 +963,9 @@ impl NaiveDateTime {
     /// assert_eq!(dt.timezone(), Utc);
     /// ```
     #[must_use]
-    pub fn and_utc(&self) -> DateTime<Utc> {
-        Utc.from_utc_datetime(self)
+    pub const fn and_utc(&self) -> DateTime<Utc> {
+        // FIXME: use `DateTime::from_naive_utc_and_offset` when our MSRV is 1.61+.
+        DateTime::from_naive_utc(*self)
     }
 
     /// The minimum possible `NaiveDateTime`.