On 12/25/25 4:05 AM, Zshengbin91 wrote:
> I'm using postgres, and I found a potentially ambiguous code comment in
> the `CreateFunction` function. `returnsSet` is a boolean variable, and
> to be consistent with the subsequent error message, I believe the
> comment should be modified to match the error message. "reject ROWS if
> it's not returnsSet" should be changed to "reject ROWS if it does not
> return a set" Therefore, I've attached a patch to address this issue.
Hm,
I personally don't really think your change improves anything. Either
version works just fine and I have a mild preferences for the original
comment. Honestly if it was me writing the comment I would probably have
entirely left out the part about rejecting since that is obvious from
just reading the code but now that it is there it might as well stay.
Andreas