Fix misuse of foreach_delete_current().
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 18 Mar 2021 23:24:22 +0000 (19:24 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 18 Mar 2021 23:24:22 +0000 (19:24 -0400)
commit1d581ce7129d7a33cd4ad27f8f246abfa1fd2db9
treee61e04b6ec6f070b41f38f0b267c7179a42e6696
parentbe45be9c33a85e72cdaeb9967e9f6d2d00199e09
Fix misuse of foreach_delete_current().

Our coding convention requires this macro's result to be assigned
back to the original List variable.  In this usage, since the
List could not become empty, there was no actual bug --- but
some compilers warned about it.  Oversight in be45be9c3.

Discussion: https://postgr.es/m/35077b31-2d62-1e31-0e2e-ddb52d590b73@enterprisedb.com
src/backend/parser/parse_agg.c