forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitafa178e
committed
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 madethis 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, andperhaps it should throw "not implemented". But I guess thatwould 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>1 parente4cfff6 commitafa178e
File tree
2 files changed
+2
-8
lines changed- src/interfaces/jdbc/org/postgresql
- jdbc1
- jdbc2
2 files changed
+2
-8
lines changedLines changed: 1 addition & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
419 | 419 |
| |
420 | 420 |
| |
421 | 421 |
| |
422 |
| - | |
423 |
| - | |
424 |
| - | |
425 |
| - | |
| 422 | + | |
426 | 423 |
| |
427 | 424 |
| |
428 | 425 |
| |
|
Lines changed: 1 addition & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
414 | 414 |
| |
415 | 415 |
| |
416 | 416 |
| |
417 |
| - | |
418 |
| - | |
419 |
| - | |
420 |
| - | |
| 417 | + | |
421 | 418 |
| |
422 | 419 |
| |
423 | 420 |
| |
|
0 commit comments
Comments
(0)