"David G. Johnston" <david.g.johnston@gmail.com> writes:
> Decided to ping this as I don't recall or see it getting patched and the
> recent discussion regarding the prosrc field.
>> * Note: if you change the output format of this function, be careful not
>> * to break psql's rules (in \ef and \sf) for identifying the start of the
>> * function body. To wit: the function body starts on a line that begins
>> * with "AS ", and no preceding line will look like that.
>>
>> Needless to say, the SQL-function-body patch has ignored this advice
>> totally. At the very least this comment needs to be adjusted, but
>> I wonder if it's not telling us that \ef and/or \sf are broken.
Indeed, if you experiment with "\sf+" or "\ef" with a line number,
those features fail miserably for new-style SQL functions.
We could partially fix that by teaching psql to also recognize
"BEGIN" as the start of the function body. I say "partially"
because the whole point of that line-numbers feature is so that
you can figure out which line an error report is complaining about.
However, SQL functions don't report in terms of line numbers,
and never have, so maybe that's moot. It's fortunate that they
don't, because the reconstructed function text is likely to have
different line breaks than the original.
regards, tom lane
