On Sat, Mar 25, 2023 at 9:13 PM Joseph Koshakow <kosh...@gmail.com> wrote: > > On Fri, Mar 24, 2023 at 9:43 AM Ashutosh Bapat <ashutosh.bapat....@gmail.com> > wrote: > > > > You don't need to do this, but looks like we can add DAYS_PER_WEEK macro > > and > > use it here. > > I've attached a patch with this new macro. There's probably tons of > places it can be used instead of hardcoding the number 7, but I'll save > that for a future patch.
Thanks. Yes, changing other existing usages is out of scope for this patch. Looks good to me. -- Best Wishes, Ashutosh Bapat
