From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
Cc: | PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: syntax error position "CREATE FUNCTION" bug fix |
Date: | 2004-03-18 17:56:45 |
Message-ID: | 10617.1079632605@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> writes:
>> Moreover, I don't like the proposed protocol change anyway. This
>> approach only solves the problem for psql-like error reporting, namely
>> clients that are going to regurgitate a string in their error message
>> and aren't very picky about whether that string really matches the
>> original input. One of the design goals for the error reporting feature
>> is that GUI-type clients should be able to mark syntax error positions
>> by highlighting in the original input window. This proposal abandons
>> that goal.
> I cannot see your point.
> Any GUI can take advantage of the returned string to show it in a window
> with fancy colors and do any hilighting around the position.
But it cannot (easily) match it up with the *original input* and put the
cursor in the right place in the *input* window. You are envisioning
something that's no better than psql with window borders. My idea of a
GUI syntax error report is something that puts my editing cursor in the
right place.
> I've implemented the stuff for psql (with your help btw), but I cannot see
> why it couldn't be used with other interfaces?
It's not the right thing for other interfaces. If it were the right
thing for all interfaces, we'd have put the functionality in the backend
to begin with.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2004-03-18 18:23:14 | Re: fsync method checking |
Previous Message | Tom Lane | 2004-03-18 17:51:03 | Re: Further thoughts about warning for costly FK checks |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2004-03-18 19:00:29 | Re: syntax error position "CREATE FUNCTION" bug fix |
Previous Message | Fabien COELHO | 2004-03-18 16:51:45 | Re: syntax error position "CREATE FUNCTION" bug fix |