Re: [PATCH] Documentation: Fix minor grammatical and formatting issues - Mailing list pgsql-hackers

From David Rowley
Subject Re: [PATCH] Documentation: Fix minor grammatical and formatting issues
Date
Msg-id CAApHDvqbSJQC0k+-duXO-6CBg6k5C9Wda+fFEUvxmtD=gxrfNQ@mail.gmail.com
Whole thread Raw
In response to [PATCH] Documentation: Fix minor grammatical and formatting issues  (Noboru Saito <noborusai@gmail.com>)
List pgsql-hackers
On Mon, 21 Apr 2025 at 15:06, Noboru Saito <noborusai@gmail.com> wrote:
> 1. Remove unnecessary blank lines (blankline.diff)

Looks good.

> 2. Fix repeated "to to" in several command reference files (toto.diff)

-        Specifies the name of the database to connect to to discover which
+        Specifies the name of the database to connect to discover which

What's there is acceptable English, but it's maybe confusing to a
non-native speaker and might be worth adjusting so it's easier to
parse. "connect to to discover" is just short for "connect to in order
to discover". If it's confusing, maybe we could rephrase the whole
sentence. How about:

"When the <option>-a</option>/<option>--all</option> is used, connect
to this database to gather the list of databases to vacuum."

> 3. Fix period instead of comma in enumeration (comma.diff)

Looks good.

> 4. Standardize inline tags in libpq documentation (tagmiss.diff)

Looks good.

> 5. Remove redundant "that that" constructions (thatthat.diff)

I don't see anything wrong here. [1] does a better job of explaining
#5 than I can.

David

[1] https://english.stackexchange.com/questions/492349/is-my-use-of-that-that-in-this-sentence-gramaticaly-correct



pgsql-hackers by date:

Previous
From: Steve Chavez
Date:
Subject: Re: Allow database owners to CREATE EVENT TRIGGER
Next
From: "David G. Johnston"
Date:
Subject: Re: Allow database owners to CREATE EVENT TRIGGER