Date: Thu, 23 Oct 2025 11:06:59 +0900 (JST)
Message-Id: <20251023.110659.292357580714448419.ishii@postgresql.org>
To: dgrowleyml@gmail.com, li.evan.chao@gmail.com
Cc: tgl@sss.pgh.pa.us, pgsql-hackers@lists.postgresql.org,
 pramsey@cleverelephant.ca, ojford@gmail.com, peter@eisentraut.org,
 krasiyan@gmail.com, vik@postgresfriends.org, andrew@tao11.riddles.org.uk,
 david@fetter.org
Subject: Re: Add RESPECT/IGNORE NULLS and FROM FIRST/LAST options
From: Tatsuo Ishii <ishii@postgresql.org>
In-Reply-To: <20251022.144911.1367366293633639433.ishii@postgresql.org>
References: <20251020.125847.997839131426057290.ishii@postgresql.org>
	<CAApHDvqyCEGc7taz3OiF-LMsWmUVhgmGErWR9rH+wtFU-U-JCg@mail.gmail.com>
	<20251022.144911.1367366293633639433.ishii@postgresql.org>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=iso-2022-jp
Content-Transfer-Encoding: 7bit
Archived-At: 
 <https://www.postgresql.org/message-id/20251023.110659.292357580714448419.ishii%40postgresql.org>
Precedence: bulk

>> On Mon, 20 Oct 2025 at 16:59, Tatsuo Ishii <ishii@postgresql.org> wrote:
>>>
>>> > A very trivial commit:
>>> >
>>> > ```
>>> > +     else
>>> > +
>>> > +             /*
>>> > +              * For other cases we have no idea what position of row callers would
>>> > +              * fetch next time. Also for relpos < 0 case (we go backward), we
>>> > +              * cannot set mark either. For those cases we always set mark at 0.
>>> > +              */
>>> > +             mark_pos = 0;
>>> > ```
>>> >
>>> > The empty line after “else” is not needed.
>>>
>>> That was added by pgindent.
>> 
>> If it's written down somewhere, I can't find it, but the rule we
>> normally follow here is; don't use braces if the code block has a
>> single statement without any comments that appear on a separate line.
>> Otherwise, use braces.
> 
> Oh ok, I didn't know that.
> 
>> Since your comments are not on the same line as the statement, it
>> should have braces. I imagine that's why pgindent is "acting weird".
> 
> Attached is a trivial patch to follow the rule.

Patch pushed. Thanks.
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp