Re: backend/nodes cleanup: Move loop variables definitions into for statement

[Peter Eisentraut]

On 25.11.25 04:46, Chao Li wrote:

On Tue, Nov 25, 2025 at 7:38 AM Chao Li <li.evan.c...@gmail.com <mailto:li.evan.c...@gmail.com>> wrote:

    On Mon, Nov 24, 2025 at 11:00 PM Peter Eisentraut
    <pe...@eisentraut.org <mailto:pe...@eisentraut.org>> wrote:

        On 19.11.25 11:32, Chao Li wrote:
         > On Wed, Nov 19, 2025 at 5:36 PM Peter Eisentraut
        <pe...@eisentraut.org <mailto:pe...@eisentraut.org>
         > <mailto:pe...@eisentraut.org <mailto:pe...@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