Return-Path: Delivered-To: apmail-db-derby-dev-archive@www.apache.org Received: (qmail 89122 invoked from network); 9 Jun 2005 21:20:48 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 9 Jun 2005 21:20:48 -0000 Received: (qmail 91278 invoked by uid 500); 9 Jun 2005 21:20:47 -0000 Delivered-To: apmail-db-derby-dev-archive@db.apache.org Received: (qmail 91255 invoked by uid 500); 9 Jun 2005 21:20:46 -0000 Mailing-List: contact derby-dev-help@db.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: List-Id: Reply-To: "Derby Development" Delivered-To: mailing list derby-dev@db.apache.org Received: (qmail 91234 invoked by uid 99); 9 Jun 2005 21:20:46 -0000 X-ASF-Spam-Status: No, hits=0.1 required=10.0 tests=HTML_50_60,HTML_MESSAGE,RCVD_BY_IP,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (hermes.apache.org: domain of m.v.lunteren@gmail.com designates 64.233.162.206 as permitted sender) Received: from zproxy.gmail.com (HELO zproxy.gmail.com) (64.233.162.206) by apache.org (qpsmtpd/0.28) with ESMTP; Thu, 09 Jun 2005 14:20:44 -0700 Received: by zproxy.gmail.com with SMTP id 8so215613nzo for ; Thu, 09 Jun 2005 14:20:39 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:in-reply-to:mime-version:content-type:references; b=sniOu9PNxua0mgZaFMvo2qO6sfs1r6WmT2eP0QIvZ77OB83dOt7w6T020nthr+NEHhkZhBiMIAwJQlQ3NvsIkGR/7+TrJqDA5jLJ/7CPY0TSjZaCnsb7XDpR02fzneJzqemdujs50gLIKPJFFx2w2jcwGTZ+1BVvTBANyx4RNAk= Received: by 10.36.2.20 with SMTP id 20mr779569nzb; Thu, 09 Jun 2005 14:20:39 -0700 (PDT) Received: by 10.36.13.17 with HTTP; Thu, 9 Jun 2005 14:20:38 -0700 (PDT) Message-ID: Date: Thu, 9 Jun 2005 14:20:39 -0700 From: Myrna van Lunteren Reply-To: Myrna van Lunteren To: Derby Development Subject: Re: [PATCH] (DERBY-276) ResultSet.relative(int row) cannot be called when the cursor is not positioned on a row. In-Reply-To: Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_1202_13307247.1118352039008" References: <20050527123139.GA30208@atum01.norway.sun.com> <20050608213229.GA6409@barbar.sun.com> X-Virus-Checked: Checked X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N ------=_Part_1202_13307247.1118352039008 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Oh - maybe better to attach your next patch to the bug? Myrna On 6/9/05, Myrna van Lunteren wrote:=20 >=20 > On 6/8/05, Bernt M. Johnsen wrote:=20 > >=20 > > >>>>>>>>>>>> Bernt M. Johnsen wrote (2005-05-27 14:31:39): > > > Could anybody review this?=20 > >=20 > > Well? > >=20 > > > This patch will make relative() allowed when the cursor is before > > > first or after last. It will also let absolute(0) be equivalent to > > > beforeFirst(). > > [..snip..] >=20 > Well, well!=20 > I applied the patch without trouble (I feared it would be out of date=20 > already :-) ), built without trouble. It all looks reasonable to me. > However, your patch doesn't include a modified master for scrollCursors2? > Did you run derbyall? > Could this change create trouble for any existing applications? i.e. Wha= t=20 > if someone has programmed an app to do relative(1) in a loop, breaking ou= t=20 > when the exception is hit? > Thx, > Myrna > ------=_Part_1202_13307247.1118352039008 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline
Oh - maybe better to attach your next patch to the bug?
Myrna


 
On 6/9/05, M= yrna van Lunteren <m.v.lun= teren@gmail.com> wrote:
On 6/8/05, Bernt M. Johnsen < Bernt.Johnsen@sun.com> wrote:
>>>&g= t;>>>>>>>> Bernt M. Johnsen wrote (2005-05-27 14:31= :39):
> Could anybody review this?

Well?

> This patch wi= ll make relative() allowed when the cursor is before
> first or after= last. It will also let absolute(0) be equivalent to
> beforeFirst().
[..snip..]
 
 
Well, well!
I applied the patch without trouble (I feared it would be out of = date already :-) ), built without trouble. It all looks reasonable to me.
However, your patch doesn't include a modified master for scrollCursor= s2?
Did you run derbyall?
 
Could this change create trouble for any existing applications? i.e. W= hat if someone has programmed an app to do relative(1) in a loop, breaking = out when the exception is hit?
 
Thx,
Myrna

------=_Part_1202_13307247.1118352039008--