Re: [PATCH] Documentation: Fix minor grammatical and formatting issues

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