Re: Remove redundant assignment in CreateWorkExprContext

From: Andreas Karlsson <andreas(at)proxel(dot)se>
To: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Remove redundant assignment in CreateWorkExprContext
Date: 2026-01-14 07:13:17
Message-ID: 08b563af-1005-41a1-8ae1-a6aad41ad158@proxel.se
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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

Attachment Content-Type Size
v2-0001-Remove-redundant-assignemnt-in-CreateWorkExprCont.patch text/x-patch 974 bytes

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Steven Niu 2026-01-14 07:22:35 Re: [PATCH] check kernel version for io_method
Previous Message Alena Vinter 2026-01-14 07:11:45 Re: Timeline switching with partial WAL records can break replica recovery