Hi Pavel,
In xfunc.sgml, I came across a function example (for use of VARIADIC in polymorphic functions),
where the function name is concat(): (in the manual: 35.4.10. Polymorphic SQL Functions).
Although that is not strictly wrong, it seems better to change that name when concat goes into
core, as seems to be the plan.
If you agree, it seems best to include this change in your patch and change that example
function's name when the stringfunc patch gets applied.
Erik Rijkers