diff options
author | Bruce Momjian <bruce@momjian.us> | 2001-09-06 18:26:37 +0000 |
---|---|---|
committer | Bruce Momjian <bruce@momjian.us> | 2001-09-06 18:26:37 +0000 |
commit | afa178e7e2118800d9723a0608eb1dc8a4beef6f (patch) | |
tree | 4d386eef38b06917fec0bc378e0ca578078e2893 /src/backend/executor/execProcnode.c | |
parent | e4cfff6520b5f3825194e3711f1b2ea31428ba30 (diff) | |
download | postgresql-afa178e7e2118800d9723a0608eb1dc8a4beef6f.tar.gz postgresql-afa178e7e2118800d9723a0608eb1dc8a4beef6f.zip |
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>
Diffstat (limited to 'src/backend/executor/execProcnode.c')
0 files changed, 0 insertions, 0 deletions