Re: errposition - Mailing list pgsql-hackers

From Tom Lane
Subject Re: errposition
Date
Msg-id 921810.1778214774@sss.pgh.pa.us
Whole thread
In response to errposition  (John Naylor <johncnaylorls@gmail.com>)
List pgsql-hackers
John Naylor <johncnaylorls@gmail.com> writes:
> In doc/src/sgml/sources.sgml under "available auxiliary routines for
> ereport", we have

> "errposition(int cursorpos) specifies the textual location of an error
> within a query string. Currently it is only useful for errors detected
> in the lexical and syntactic analysis phases of query processing."

> ...but I don't see that function used that way. It seems this should
> be parser_errposition instead. I'll go make it so unless someone tells
> me I'm missing something.

errposition is the underlying function exposed by elog.c, and
that is what this documentation is intending to document.
parser_errposition is a wrapper that is specific to usages in
src/backend/parser/.  It might be (I didn't check) that all existing
calls to errposition go through parser_errposition, but that seems
like a happenstance not a reason to contort the documentation.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: Re: [Patch] Omit virtual generated columns from test_decoding output
Next
From: Etsuro Fujita
Date:
Subject: Re: Bug: trailing comma syntax error in postgres_fdw fetch_attstats()