RE: fix typo in reorderbuffer.c

From: "Hou, Zhijie" <houzj(dot)fnst(at)cn(dot)fujitsu(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: RE: fix typo in reorderbuffer.c
Date: 2021-01-27 07:15:41
Message-ID: ade6bdf4a588488ab0669393dca8af15@G08CNEXMBPEKD05.g08.fujitsu.local
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> >> Combocid's ==>> Combocids
> >
> > Ping again, just in case it’s missed.
>
> There are in the comments "ComboCIDs", "combocids" and "ComboCids" on top
> of the existing Combocid's. Your patch introduces a fourth way of writing
> that. It may be better to just have one way to govern them all.

Hi Michael,

Thanks for the review.

Actually, "Combocid's" in the patch is the first word of the sentence, so the first char is uppercase(may be not a new style).

I agree that it’s better to have a common way, but some different style of combocid follow the variable or functionname[1].
We may need to change the var name or function name too.

Personally , I prefer "combocids".
But do you think we can change that in a separate patch apart from this typo patch ?

[1]:
void
SerializeComboCIDState(Size maxsize, char *start_address)

static int usedComboCids = 0; /* number of elements in comboCids */
...

Best regards,
houzj

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Masahiko Sawada 2021-01-27 07:20:02 Re: Is Recovery actually paused?
Previous Message Peter Geoghegan 2021-01-27 07:09:14 Re: FailedAssertion in heap_index_delete_tuples at heapam.c:7220