Unify day-of-week calculation (#6368)

Weekdays are not calender-specific, and the current code is overly
complex.

Author: robertbastian

components/calendar/src/cal/coptic.rs

@@ -131,10 +131,6 @@ impl Calendar for Coptic {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(Coptic.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &());
     }

components/calendar/src/cal/dangi.rs

@@ -242,10 +242,6 @@ impl Calendar for Dangi {
         date.0 .0.day_of_year()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> crate::types::Weekday {
-        self.date_to_iso(date).day_of_week()
-    }
-
     fn any_calendar_kind(&self) -> Option<crate::AnyCalendarKind> {
         Some(crate::any_calendar::IntoAnyCalendar::kind(self))
     }

components/calendar/src/cal/ethiopian.rs

@@ -158,10 +158,6 @@ impl Calendar for Ethiopian {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(self.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &());
     }

components/calendar/src/cal/hebrew.rs

@@ -497,10 +497,10 @@ mod tests {
         let cal = Hebrew::new();
         let era = "am";
         let month_code = MonthCode(tinystr!(4, "M01"));
-        let dt = cal.date_from_codes(Some(era), 3760, month_code, 1).unwrap();
+        let dt = Date::try_new_from_codes(Some(era), 3760, month_code, 1, cal).unwrap();
 
         // Should be Saturday per:
         // https://www.hebcal.com/converter?hd=1&hm=Tishrei&hy=3760&h2g=1
-        assert_eq!(6, cal.day_of_week(&dt) as usize);
+        assert_eq!(6, dt.day_of_week() as usize);
     }
 }

components/calendar/src/cal/hijri.rs

@@ -484,10 +484,6 @@ impl Calendar for HijriObservational {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(self.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &self.precomputed_data())
     }
@@ -839,10 +835,6 @@ impl Calendar for HijriCivil {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(self.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &())
     }
@@ -1020,10 +1012,6 @@ impl Calendar for HijriTabular {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(self.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &())
     }

components/calendar/src/cal/indian.rs

@@ -156,10 +156,6 @@ impl Calendar for Indian {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(Indian.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &());
     }

components/calendar/src/cal/iso.rs

@@ -114,54 +114,6 @@ impl Calendar for Iso {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        // For the purposes of the calculation here, Monday is 0, Sunday is 6
-        // ISO has Monday=1, Sunday=7, which we transform in the last step
-
-        // The days of the week are the same every 400 years
-        // so we normalize to the nearest multiple of 400
-        let years_since_400 = date.0.year.rem_euclid(400);
-        debug_assert!(years_since_400 >= 0); // rem_euclid returns positive numbers
-        let years_since_400 = years_since_400 as u32;
-        let leap_years_since_400 = years_since_400 / 4 - years_since_400 / 100;
-        // The number of days to the current year
-        // Can never cause an overflow because years_since_400 has a maximum value of 399.
-        let days_to_current_year = 365 * years_since_400 + leap_years_since_400;
-        // The weekday offset from January 1 this year and January 1 2000
-        let year_offset = days_to_current_year % 7;
-
-        // Corresponding months from
-        // https://en.wikipedia.org/wiki/Determination_of_the_day_of_the_week#Corresponding_months
-        let month_offset = if Self::is_leap_year(date.0.year, ()) {
-            match date.0.month {
-                10 => 0,
-                5 => 1,
-                2 | 8 => 2,
-                3 | 11 => 3,
-                6 => 4,
-                9 | 12 => 5,
-                1 | 4 | 7 => 6,
-                _ => unreachable!(),
-            }
-        } else {
-            match date.0.month {
-                1 | 10 => 0,
-                5 => 1,
-                8 => 2,
-                2 | 3 | 11 => 3,
-                6 => 4,
-                9 | 12 => 5,
-                4 | 7 => 6,
-                _ => unreachable!(),
-            }
-        };
-        let january_1_2000 = 5; // Saturday
-        let day_offset = (january_1_2000 + year_offset + month_offset + date.0.day as u32) % 7;
-
-        // We calculated in a zero-indexed fashion, but ISO specifies one-indexed
-        types::Weekday::from((day_offset + 1) as usize)
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &());
     }

components/calendar/src/cal/julian.rs

@@ -122,10 +122,6 @@ impl Calendar for Julian {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(Julian.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &());
     }

components/calendar/src/cal/persian.rs

@@ -123,10 +123,6 @@ impl Calendar for Persian {
         date.0.days_in_month()
     }
 
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        Iso.day_of_week(self.date_to_iso(date).inner())
-    }
-
     fn offset_date(&self, date: &mut Self::DateInner, offset: DateDuration<Self>) {
         date.0.offset_date(offset, &())
     }

components/calendar/src/calendar.rs

@@ -47,10 +47,6 @@ pub trait Calendar {
     /// Count the number of days in a given month, specified by providing a date
     /// from that year/month
     fn days_in_month(&self, date: &Self::DateInner) -> u8;
-    /// Calculate the day of the week and return it
-    fn day_of_week(&self, date: &Self::DateInner) -> types::Weekday {
-        self.date_to_iso(date).day_of_week()
-    }
     // fn week_of_year(&self, date: &Self::DateInner) -> u8;
 
     #[doc(hidden)] // unstable