Re: Another jdbc2 ResultSet patch

Applied. Thanks.

> This patch fixes the 0-based/1-based result set indexing problem for
> absolute. It also makes it more compliant with the interface
> specification in Sun's documentation;
>
> 1. absolute(0) should throw an exception.
> 2. absolute(>num-records) should set the current row to after the last
> record in addition to returning false.
> 3. absolute(<num-records) should set the current row to before the first
> record in addition to returning false.
>
> These operations in the existing code just return false and don't change
> current_row.
>
> These changes required a minor change to relative(int) since it calls
> absolute(int)
>
> The attached patch is against the cvs repository tree as of this morning.
>
> Also, who is in charge of maintaining the jdbc driver? I'm working on
> getArray for the jdbc2 driver, but it's going to require three more
> classes to be added to the driver, and thus three more source files
> in the repository. Is there someone I can contact directly to ask about
> this?
>
> Thanks,
>
> ----------------------------------------------------------------
> Travis Bauer | CS Grad Student | IU |www.cs.indiana.edu/~trbauer
> ----------------------------------------------------------------
Content-Description:

[ Attachment, skipping... ]

--
Bruce Momjian | http://www.op.net/~candle
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026