Thread: constify arguments of copy_file() and copydir()

constify arguments of copy_file() and copydir()

From
Nathan Bossart
Date:
I've attached a patch for $SUBJECT, which allows us to remove a use of the
unconstify macro in basic_archive.  This is just a pet peeve, but maybe it
bothers others, too.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

Attachment

Re: constify arguments of copy_file() and copydir()

From
Andrew Dunstan
Date:
On 2023-01-14 Sa 18:11, Nathan Bossart wrote:
> I've attached a patch for $SUBJECT, which allows us to remove a use of the
> unconstify macro in basic_archive.  This is just a pet peeve, but maybe it
> bothers others, too.


I don't like using unconstify where it can be avoided, so this looks
reasonable to me.


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com




Re: constify arguments of copy_file() and copydir()

From
Nathan Bossart
Date:
On Sun, Jan 15, 2023 at 08:23:13AM -0500, Andrew Dunstan wrote:
> On 2023-01-14 Sa 18:11, Nathan Bossart wrote:
>> I've attached a patch for $SUBJECT, which allows us to remove a use of the
>> unconstify macro in basic_archive.  This is just a pet peeve, but maybe it
>> bothers others, too.
> 
> I don't like using unconstify where it can be avoided, so this looks
> reasonable to me.

Thanks.  Added to the next commitfest:

    https://commitfest.postgresql.org/42/4126/

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com



Re: constify arguments of copy_file() and copydir()

From
Michael Paquier
Date:
On Sun, Jan 15, 2023 at 08:13:28AM -0800, Nathan Bossart wrote:
> On Sun, Jan 15, 2023 at 08:23:13AM -0500, Andrew Dunstan wrote:
>> On 2023-01-14 Sa 18:11, Nathan Bossart wrote:
>>> I've attached a patch for $SUBJECT, which allows us to remove a use of the
>>> unconstify macro in basic_archive.  This is just a pet peeve, but maybe it
>>> bothers others, too.
>>
>> I don't like using unconstify where it can be avoided, so this looks
>> reasonable to me.

+1.
--
Michael

Attachment

Re: constify arguments of copy_file() and copydir()

From
Michael Paquier
Date:
On Mon, Jan 16, 2023 at 10:53:40AM +0900, Michael Paquier wrote:
> +1.

While I don't forget about this thread..  Any objections if I were to
apply that?
--
Michael

Attachment

Re: constify arguments of copy_file() and copydir()

From
Peter Eisentraut
Date:
On 17.01.23 07:24, Michael Paquier wrote:
> On Mon, Jan 16, 2023 at 10:53:40AM +0900, Michael Paquier wrote:
>> +1.
> 
> While I don't forget about this thread..  Any objections if I were to
> apply that?

Looks good to me.




Re: constify arguments of copy_file() and copydir()

From
Michael Paquier
Date:
On Tue, Jan 17, 2023 at 08:23:49PM +0100, Peter Eisentraut wrote:
> Looks good to me.

Thanks, done.
--
Michael

Attachment

Re: constify arguments of copy_file() and copydir()

From
Nathan Bossart
Date:
On Wed, Jan 18, 2023 at 09:05:52AM +0900, Michael Paquier wrote:
> Thanks, done.

Thanks!

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com