On Sat, Mar 25, 2023 at 9:13 PM Joseph Koshakow <koshy44@gmail.com> wrote:
>
> On Fri, Mar 24, 2023 at 9:43 AM Ashutosh Bapat <ashutosh.bapat.oss@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