"David G. Johnston" <david.g.johns...@gmail.com> writes:
> The omission of the "OUT" parameter mode seems intentional since at present
> our procedures do not support OUT mode parameters.

Um, I just created one.  I think this *used* to be true, and this bit of
the docs didn't get fixed.  If I back-patch this, I'll have to research
when it changed.

> Instead of "The difference" or "One difference" I would suggest: "However,
> a procedure does not return a value, so there is no return type
> declaration; though a procedure can declare INOUT (but not plain OUT)
> parameters."

Not sure if that's an improvement.

> Relocating the links to the description instead of usage is good.  The
> additional procedure link after the examples seems redundant, particularly
> as the linked to location doesn't actually have more examples.

I was modeling that on the existing pattern in create_function.sgml,
which has similar verbiage in the EXAMPLES section.  But I suppose
we could drop that if we have a link in the description section.

                        regards, tom lane


Reply via email to