Re: Remove unneccessary memory initialization in planner.c

Aleksander Alekseev <aleksander@tigerdata.com>

From: Aleksander Alekseev <aleksander@tigerdata.com>
To: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Cc: Chao Li <li.evan.chao@gmail.com>
Date: 2025-12-08T12:17:25Z
Lists: pgsql-hackers
Hi Chao,

> "gd" is allocated by palloc0, so all members have been zero initialized, thus explicitly assigning NULL to the 4 members is unnecessary. I don't think that has much runtime impact, but from a readability perspective, a potential question is that, "gd" has more fields, why only set 0 to these 4 members redundantly?
>
> To avoid the potential confusion, I think we'd better delete these initializations.

Good find. I agree that this code is redundant. Or alternatively we
should do palloc() and assign all the initial values explicitly.

-- 
Best regards,
Aleksander Alekseev