Re: Patch for jdbc2 ResultSet.java - Mailing list pgsql-jdbc

From Joseph Shraibman
Subject Re: Patch for jdbc2 ResultSet.java
Date
Msg-id 3B97E026.3010506@selectacast.net
Whole thread Raw
In response to Patch for jdbc2 ResultSet.java  (Joseph Shraibman <jks@selectacast.net>)
Responses Re: Patch for jdbc2 ResultSet.java  (Bruce Momjian <pgman@candle.pha.pa.us>)
Re: Patch for jdbc2 ResultSet.java  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-jdbc

Dave Harkness wrote:
> At 12:41 PM 9/5/2001, Joseph Shraibman wrote:
>
>> new patch:
>
>
> There still seems to be an error with the same if-block.
>
> !       if (index>=-rows.size())
> !       internalIndex=rows.size()+index;
>
> becomes
>
> !       if (index > -rows_size)
> !       internalIndex = rows_size+index;
>
> Note that the original used >=, not >.

Argh!  I need a better screen. :(
>
> Also, why is the first edit being done? Granted it's faster in that it
> doesn't null out the array of rows, but won't that have other effects?

No.  Nulling out the array is pointless.




--
Joseph Shraibman
jks@selectacast.net
Increase signal to noise ratio.  http://www.targabot.com


pgsql-jdbc by date:

Previous
From: Rene Pijlman
Date:
Subject: Re: jdbc ResultSetMetaData::isWritable()
Next
From: Bruce Momjian
Date:
Subject: Re: jdbc ResultSetMetaData::isWritable()