Re: minor MERGE cleanups

2022-04-20 Thread Alvaro Herrera
On 2022-Apr-20, Michael Paquier wrote: > On Tue, Apr 19, 2022 at 03:45:22PM +0200, Alvaro Herrera wrote: > > I expect these fixups in new code should be uncontroversial. > > The whole set looks rather sane to me. Thank you, I have pushed them. -- Álvaro HerreraBreisgau, Deutschland —

Re: minor MERGE cleanups

2022-04-19 Thread Michael Paquier
On Tue, Apr 19, 2022 at 03:45:22PM +0200, Alvaro Herrera wrote: > I expect these fixups in new code should be uncontroversial. The whole set looks rather sane to me. -- Michael signature.asc Description: PGP signature

minor MERGE cleanups

2022-04-19 Thread Alvaro Herrera
There's a complain from Coverity about outer_plan being referenced while possibly NULL, which can be silenced by using an existing local variable. 0001 does that. 0002 and 0003 are unrelated: in the former, we avoid creating a separate local variable planSlot when we can just refer to the eponymo