Re: Add RESPECT/IGNORE NULLS and FROM FIRST/LAST options

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.

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".

David