From: Calvin Wan <calvinwan@google.com>
Date: Wed, 28 Jun 2023 16:44:59 GMT
Subject: Re: [RFC PATCH 6/8] pager: remove pager_in_use()
Message-ID: <CAFySSZBBAatvBmfwn9vc=v2hdWX_0Q4g4txqLOwtqiFZFW7uiA@mail.gmail.com>
In-Reply-To: <kl6lpm5fr19f.fsf@chooglen-macbookpro.roam.corp.google.com>

> Glen Choo <chooglen@google.com> writes:
>
> >                      Could we add a is_pager/pager_in_use to that
> > function and push the pager.h dependency upwards?
>
> Bleh, I meant "Could we add a new is_pager/pager_in_use parameter to
> that function?"

Refactoring the function signature to:

parse_date_format(const char *format, struct date_mode *mode, int pager_in_use)

as you suggested is a much better solution, thanks! I'll make that
change in the next reroll.