Re: Stack overflow issue - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Stack overflow issue
Date
Msg-id 20240417173528.257mnitlhncdxx23@awork3.anarazel.de
Whole thread Raw
In response to Re: Stack overflow issue  (Alexander Korotkov <aekorotkov@gmail.com>)
List pgsql-hackers
Hi,

On 2024-04-17 14:39:14 +0300, Alexander Korotkov wrote:
> On Wed, Apr 17, 2024 at 2:37 PM Alexander Korotkov <aekorotkov@gmail.com> wrote:
> > I've invested more time into polishing this.  I'm intended to push
> > this.  Could you, please, take a look before?
> 
> Just after sending this I spotted a typo s/untill/until/.  The updated
> version is attached.

Nice, I see you moved the code back to "where it was", the diff to 16 got
smaller this way.


> +    /*
> +     * Repeatedly call CommitTransactionCommandInternal() until all the work
> +     * is done.
> +     */
> +    while (!CommitTransactionCommandInternal());

Personally I'd use
{
}
instead of just ; here. The above scans weirdly for me. But it's also not
important.

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: documentation structure
Next
From: Thomas Munro
Date:
Subject: Re: Solaris tar issues, or other reason why margay fails 010_pg_basebackup?