Gavin, have you answered these issues brought up about the patch?

---------------------------------------------------------------------------

Tom Lane wrote:
> Gavin Sherry <[EMAIL PROTECTED]> writes:
> > Attached is a small patch to scan.l for consideration. It hands
> > yyerror() the position in the query string of the token which caused a
> > parse error.
> 
> Isn't that the hard way to do it?  Seems like you could just subtract
> scanbuf from the error pointer, instead of adding overhead to the basic
> lex loop.
> 
> Things that need to be decided (and documented somewhere):
> 
> Is the number an offset (counted from 0) or an index (counted from 1)?
> Which end of the token does it point at?  Can the message be phrased
> so as to make it reasonably clear what the number is?
> 
> 
> A related change I'd been meaning to make is to get it to say
>       parse error at end of input
> when that's the case, rather than the rather useless
>       parse error at or near ""
> that you get now.  I'd still be inclined to just say "end of input"
> in that case, and not bother with a character count.
> 
>                       regards, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]
> 

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  [EMAIL PROTECTED]               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 6: Have you searched our list archives?

http://archives.postgresql.org

Reply via email to