> 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/






Reply via email to