> On Jan 14, 2026, at 15:13, Andreas Karlsson <andreas@proxel.se> 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,
soI’m happy to leave the final decision to the committers.
Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/