| From: | Joseph Shraibman <jks(at)selectacast(dot)net> |
|---|---|
| To: | Dave Harkness <daveh(at)MEconomy(dot)com> |
| Cc: | Barry Lind <barry(at)xythos(dot)com>, pgsql-patches(at)postgresql(dot)org, pgsql-jdbc(at)postgresql(dot)org |
| Subject: | Re: Patch for jdbc2 ResultSet.java |
| Date: | 2001-09-06 20:44:22 |
| Message-ID: | 3B97E026.3010506@selectacast.net |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-jdbc pgsql-patches |
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(at)selectacast(dot)net
Increase signal to noise ratio. http://www.targabot.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Falconer | 2001-09-06 21:30:58 | DatabaseMetadata problems |
| Previous Message | Bruce Momjian | 2001-09-06 20:43:51 | Re: jdbc ResultSetMetaData::isWritable() |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2001-09-06 20:45:25 | Re: IPv6 Support for INET/CIDR types. |
| Previous Message | Bruce Momjian | 2001-09-06 20:07:46 | Re: JDBC patch procedures (Re: [PATCHES] Patch for jdbc2 ResultSet.java) |