On Wed, 2024-05-22 at 18:34 +0300, Igor Gnatyuk wrote: > fix_doc_raise_v3-bios.patch. Please check it out.
I think the patch is fine. > + <para> > + <replaceable class="parameter">condition_name</replaceable> and > + <replaceable class="parameter">sqlstate</replaceable> specify > + error condition name and the five-character SQLSTATE code respectively. > + See <xref linkend="errcodes-appendix"/> for more information. > + </para> There should be a comma before "respectively". > You can attach additional information to the error report by writing > <literal>USING</literal> followed by <replaceable > class="parameter">option</replaceable> = <replaceable > - class="parameter">expression</replaceable> items. Each > + class="parameter">expression</replaceable> or > + <replaceable class="parameter">option</replaceable> := > + <replaceable class="parameter">expression</replaceable> > + items, where > <replaceable class="parameter">expression</replaceable> can be any > string-valued expression. I think that is unnecessarily verbose. The original wording was fine; everybody can see from the syntax diagram that you can also use := But I won't fight over it. > + In the <command>RAISE</command> command syntax with > + <replaceable class="parameter">condition_name</replaceable> or > + <replaceable class="parameter">sqlstate</replaceable> you can > + additionally use the <literal>USING</literal> clause too. > + A variation of the example above: I think that the final sentence should be more complete. Suggestions: Here is a variation of the above example: A variation of the above example is: Yours, Laurenz Albe
