Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name - Mailing list pgsql-hackers

From Yurii Rashkovskii
Subject Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name
Date
Msg-id CA+RLCQy+rP+KdhjvNkUXwToVwO=pWrhDFqqaMKpmRpGKh+Ns8w@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name  (Nathan Bossart <nathandbossart@gmail.com>)
Responses Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name  (Nathan Bossart <nathandbossart@gmail.com>)
List pgsql-hackers
Nathan,

On Mon, Jul 3, 2023 at 3:08 PM Nathan Bossart <nathandbossart@gmail.com> wrote:
On Sun, Jul 02, 2023 at 04:37:52PM -0700, Yurii Rashkovskii wrote:
> Thank you for revising the patch. While this is relatively minor, I think
> it should be set to MAXPGPATH directly to clarify their relationship.

Committed.  I set the size to MAXPGPATH directly instead of inventing a new
macro with the same value.

Great, thank you! The reason I was leaving the other constant in place to make upgrading extensions trivial (so that they don't need to adjust for this), but if you think this is a better way, I am fine with it.
 
--
Y.

pgsql-hackers by date:

Previous
From: Richard Guo
Date:
Subject: Re: Improve list manipulation in several places
Next
From: James Coleman
Date:
Subject: pgindent (probably my missing something obvious)