|From:||Michael Paquier <michael(at)paquier(dot)xyz>|
|To:||David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>|
|Cc:||Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Andres Freund <andres(at)anarazel(dot)de>, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>|
|Subject:||Re: Passing CopyMultiInsertInfo structure to CopyMultiInsertInfoNextFreeSlot()|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On Tue, May 14, 2019 at 01:19:30PM +1200, David Rowley wrote:
> When I wrote the code I admit that I was probably wearing my
> object-orientated programming hat. I had in mind that the whole
> function series would have made a good class. Passing the
> CopyMultiInsertInfo was sort of the non-OOP equivalent to having
> this/Me/self available, as it would be for any instance method of a
> class. Back to reality, this isn't OOP, so I was wearing the wrong
> hat. I think the unused parameter should likely be removed. It's
> probably not doing a great deal of harm since the function is static
> inline and the compiler should be producing any code for the unused
> param, but for the sake of preventing confusion, it should be removed.
> Ashutosh had to ask about it, so it wasn't immediately clear what the
> purpose of it was. Since there's none, be gone with it, I say.
Sounds fair to me. This has been introduced by 86b8504, so let's see
what's Andres take.
|Next Message||Bruce Momjian||2019-05-14 02:55:01||Re: pg12 release notes|
|Previous Message||Thomas Munro||2019-05-14 02:42:18||Re: ldapbindpasswdfile|