> On Jan 14, 2026, at 15:13, Andreas Karlsson <[email protected]> wrote:
>
> On 8/21/25 5:47 AM, Chao Li wrote:
>> While discussing [1], I was reading execUtils.c, then I noticed this
>> redundant local variable assignment in CreateWorkExprContext(). The attached
>> patch fixed that.
> Nice spotted but I think your patch reduces readability. How about this?
>
> Andreas
> <v2-0001-Remove-redundant-assignemnt-in-CreateWorkExprCont.patch>
Hi Andraes,
Thank you very much for reviewing and bumping this patch. I’m fine with your
tuning; I don’t have a strong preference, so I’m happy to leave the final
decision to the committers.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/