On 12.11.25 09:00, Chao Li wrote:
While working on the other patch [1] that touched tbm_add_tuples() under
src/backend/nodes, I moved a loop variable into for statement by the way
as I knew Peter Eisentraut had done some efforts about that, see [2].
However, Peter removed that change from the patch and he said that
should belong to a separate cleanup patch. So I am following up and
moving loop variables into for statements wherever possible under src/
backend/nodes.
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).
