Thread: pgsql: Fix wrong file path in header comment

pgsql: Fix wrong file path in header comment

From
Alexander Korotkov
Date:
Fix wrong file path in header comment

Header comment of shm_mq.c was mistakenly specifying path to shm_mq.h.
It was introduced in ec9037df.  So, theoretically it could be
backpatched to 9.4, but it doesn't seem to worth it.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a01d0fa1d889cc2003e1941e8b98707c4d701ba9

Modified Files
--------------
src/backend/storage/ipc/shm_mq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


Re: pgsql: Fix wrong file path in header comment

From
Alexander Korotkov
Date:
On Wed, Jul 11, 2018 at 1:26 PM Alexander Korotkov
<akorotkov@postgresql.org> wrote:
> Header comment of shm_mq.c was mistakenly specifying path to shm_mq.h.
> It was introduced in ec9037df.  So, theoretically it could be
> backpatched to 9.4, but it doesn't seem to worth it.

I didn't backpatch, because I thought it doesn't worth it to create
buzz over single wrong path in the comment.  But, I found 12 similar
cases in our source tree.  Do you think it worth backpatching now?

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

Attachment

Re: pgsql: Fix wrong file path in header comment

From
Tom Lane
Date:
Alexander Korotkov <a.korotkov@postgrespro.ru> writes:
> I didn't backpatch, because I thought it doesn't worth it to create
> buzz over single wrong path in the comment.  But, I found 12 similar
> cases in our source tree.  Do you think it worth backpatching now?

Nah, just fix 'em in HEAD.

            regards, tom lane


Re: pgsql: Fix wrong file path in header comment

From
Alexander Korotkov
Date:
On Wed, Jul 11, 2018 at 5:36 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Alexander Korotkov <a.korotkov@postgrespro.ru> writes:
> > I didn't backpatch, because I thought it doesn't worth it to create
> > buzz over single wrong path in the comment.  But, I found 12 similar
> > cases in our source tree.  Do you think it worth backpatching now?
>
> Nah, just fix 'em in HEAD.

OK, thanks.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company