On Thu, May 19, 2022 at 11:53:22PM +0800, Julien Rouhaud wrote:
> On Thu, May 19, 2022 at 10:11:06AM -0400, Tom Lane wrote:
> > I do not like your proposed wording, as it seems way too dense.
> > Can't we avoid the parenthetical remarks (plural) inside a sub-clause?
> > You're asking the reader to keep track of about three levels of
> > interrupt.
>
> Yes, I was a bit unhappy with this wording.
>
> > Maybe just put the Windows info in a separate sentence:
> >
> > ... located at ~/.pg_service.conf</filename>.
> > On Microsoft Windows, it is located at
> > %APPDATA%\postgresql\.pg_service.conf, where %APPDATA% means the
> > Application Data subdirectory in the user's profile.
> > In either case, the location can be overridden by setting ...
> >
> > ISTR there's precedent elsewhere for doing it about like that.
>
> I'm fine with it. I still kept the default location full description in both
> places as those end up in quite different part of the documentation, which
> seems more helpful.
This time with the patch.