Request a Demo
Home   >   mailing lists

Re: backend/nodes cleanup: Move loop variables definitions into for statement - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: backend/nodes cleanup: Move loop variables definitions into for statement
Date
Msg-id 9bcbdbfe-50c4-4a49-8891-9ce229c35125@eisentraut.org
Whole thread Raw
In response to Re: backend/nodes cleanup: Move loop variables definitions into for statement  (Chao Li <li.evan.chao@gmail.com>)
List pgsql-hackers
Tree view 
On 25.11.25 04:46, Chao Li wrote:
> 
> On Tue, Nov 25, 2025 at 7:38 AM Chao Li <li.evan.chao@gmail.com 
> <mailto:li.evan.chao@gmail.com>> wrote:
> 
> 
>     On Mon, Nov 24, 2025 at 11:00 PM Peter Eisentraut
>     <peter@eisentraut.org <mailto:peter@eisentraut.org>> wrote:
> 
>         On 19.11.25 11:32, Chao Li wrote:
>          > On Wed, Nov 19, 2025 at 5:36 PM Peter Eisentraut
>         <peter@eisentraut.org <mailto:peter@eisentraut.org>
>          > <mailto:peter@eisentraut.org <mailto:peter@eisentraut.org>>>
>         wrote:
>          >
>          >
>          >
>          >     In a couple of cases, you are changing the type of the
>         loop variable
>          >     from Size to int.  I would not expect such a change in a
>         patch of this
>          >     nature.  If you have a reason to change it, please
>         explain it (but I
>          >     doubt the change is correct).
>          >
>          >
>          > That was a mistake. I fixed it in v3.
> 
>         Still not correct in outDatum().
> 
> 
>     Oops! Fixed in v4.
> 
> 
> Rebased to v5.

committed

pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Serverside SNI support in libpq
Next
From: Daniel Gustafsson
Date:
Subject: The pgperltidy diffs in HEAD