Missing comment edit
Hello.
I happend to find that the comment on formdesc is missing
pg_subscription. Please find the attached patch (I'm sure:) to
fix that .
regards,
--
Kyotaro Horiguchi
NTT Open Source Software Center
Attachments:
comment_fix_formdesc.patchtext/x-patch; charset=us-asciiDownload+2-2
Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> writes:
I happend to find that the comment on formdesc is missing
pg_subscription. Please find the attached patch (I'm sure:) to
fix that .
Hmm ... certainly, that comment is now wrong, but I'm kind of inclined
to just remove it, because it'll certainly be wrong again in future.
It's not telling you anything you can't find out with a trivial search
in the same file, so is it worth the maintenance overhead?
regards, tom lane
On 2018-02-28 19:03:01 -0500, Tom Lane wrote:
Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> writes:
I happend to find that the comment on formdesc is missing
pg_subscription. Please find the attached patch (I'm sure:) to
fix that .Hmm ... certainly, that comment is now wrong, but I'm kind of inclined
to just remove it, because it'll certainly be wrong again in future.
It's not telling you anything you can't find out with a trivial search
in the same file, so is it worth the maintenance overhead?
+1. Do you want to do the honors? If you don't and nobody comments,
I'll do so.
- Andres
Andres Freund <andres@anarazel.de> writes:
On 2018-02-28 19:03:01 -0500, Tom Lane wrote:
Hmm ... certainly, that comment is now wrong, but I'm kind of inclined
to just remove it, because it'll certainly be wrong again in future.
It's not telling you anything you can't find out with a trivial search
in the same file, so is it worth the maintenance overhead?
+1. Do you want to do the honors? If you don't and nobody comments,
I'll do so.
Done.
regards, tom lane