Re: Patch adding name for NotImplemented

Kris Jurka wrote:
>
> On Fri, 11 Feb 2005, Xavier Poinsard wrote:
>
>
>>I had some time in the train, so here is a patch adding function name to
>> notImplemented().
>
>
> It seems reather inconsistent, it uses all of the following description
> types:
> - Blob.truncate
> - setBinaryStream
> - getArray With Map
> - fetch(ResultCursor, ResultHandler, int)

Yes, I changed my mind during the process. I will try to clear some issues.

One problem is that the Exception is sometime used not for an entire
function but only when the function is used with a particular argument.
Other functions with same names are only implement with some sets of
arguments.

Would you prefer the full version :

Interface.functionname(type1,type2) ?

>
> These need to be consistent. Also some of the claims are inaccurate, for
> example Statement.setURL, Statement.getURL are actually from
> PreparedStatement and CallableStatement respectively.

It will require additional work since I will have to look not only in
the code.

>
> Finally this patch doesn't actually have a change to Driver.notImplemented
> to make it work.

Oops, I missed Driver.java.in.

>
> Kris Jurka
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org so that your
> message can get through to the mailing list cleanly
>