Re: Little cleanup of ShmemInit function names - Mailing list pgsql-hackers

From Andreas Karlsson
Subject Re: Little cleanup of ShmemInit function names
Date
Msg-id 6f27f983-da13-48e5-84c6-516a00c4922e@proxel.se
Whole thread Raw
In response to Little cleanup of ShmemInit function names  (Heikki Linnakangas <hlinnaka@iki.fi>)
Responses Re: Little cleanup of ShmemInit function names
List pgsql-hackers
On 8/28/24 7:26 PM, Heikki Linnakangas wrote:
> Hmm, I don't see the issue. It's an uncommon sentence structure, but it 
> was there before this patch, and it's correct AFAICS. If you grep for 
> "which see ", you'll find some more examples of that.

Not sure if it is correct or not but from some googling it seems to be a 
direct translation of "quod vide". I think "for which, see" would likely 
be more proper English but it is not my native language and we use 
"which see" elsewhere so we might as well be consistent and use "which see".

>> On the topic of minor language issues I think the comma below is 
>> redundant.
>>
>> "In the normal postmaster case, the shared hash tables are created here."
> 
> On second thoughts, I rearranged the sentences in the paragraph a 
> little, see attached.

Looks good!

Andreas



pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: PATCH: Add hooks for pg_total_relation_size and pg_indexes_size
Next
From: Tom Lane
Date:
Subject: Re: Little cleanup of ShmemInit function names