]> granicus.if.org Git - postgresql/commit
On Mon, 3 Sep 2001 22:01:17 -0500, you wrote:
authorBruce Momjian <bruce@momjian.us>
Thu, 6 Sep 2001 18:26:37 +0000 (18:26 +0000)
committerBruce Momjian <bruce@momjian.us>
Thu, 6 Sep 2001 18:26:37 +0000 (18:26 +0000)
commitafa178e7e2118800d9723a0608eb1dc8a4beef6f
tree4d386eef38b06917fec0bc378e0ca578078e2893
parente4cfff6520b5f3825194e3711f1b2ea31428ba30
On Mon, 3 Sep 2001 22:01:17 -0500, you wrote:
>public boolean isWritable(int column) throws SQLException
>{
>        if (isReadOnly(column))
>                return true;
>        else
>                return false;
>}

The author probably intended:

    public boolean isWritable(int column) throws SQLException
    {
        return !isReadOnly(column);
    }

And if he would have coded it this way he wouldn't have made
this mistake :-)

>hence, isWritable() will always return false. this is something
>of a problem :)

Why exactly? In a way, true is just as incorrect as false, and
perhaps it should throw "not implemented". But I guess that
would be too non-backwardly-compatible.

>let me know if i can provide further information.

Will you submit a patch?

Regards,
Ren? Pijlman <rene@lab.applinet.nl>
src/interfaces/jdbc/org/postgresql/jdbc1/ResultSetMetaData.java
src/interfaces/jdbc/org/postgresql/jdbc2/ResultSetMetaData.java