Mailing-List: contact dev-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Ant Developers List" <dev@ant.apache.org>
Received-SPF: pass (athena.apache.org: domain of peter.kitt.reilly@gmail.com
 designates 209.85.146.182 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=L+BMnj/DEnSeiqO+odX4aE4qbaY8ETzZpAp7nlyP3hk+B0STK+cWKa0N8wasAPmk9HIHVmruD+HXHJkOTNqMwZtZa0Cb16POuLvXYGIIbVtafIhTKuh8cdilc7hQgi7P5Oyn8tITIaCQ6qdVGjoYPnyTBUm25TpykZSyF8RLSi0=
Message-ID: <dffc72020711130205y76c68278k5072f05795494bfa@mail.gmail.com>
Date: Tue, 13 Nov 2007 10:05:21 +0000
From: "Peter Reilly" <peter.kitt.reilly@gmail.com>
To: "Ant Developers List" <dev@ant.apache.org>
Subject: Re: [Fwd: RE: svn commit: r594009 -
 /ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant/taskdefs/TempFile.java]
In-Reply-To: <47396FA5.6090900@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <47396FA5.6090900@gmail.com>

On Nov 13, 2007 9:34 AM, Kev Jackson <foamdino@gmail.com> wrote:
> Hi all,
>
> Some lurker ;) spotted this commit and got in touch with me.
>
> I copied over the code from the svn trunk version without looking too
> carefully (my bad), unless anyone can tell me that this is correct, I
> agree that at least semantically it's utter crap :)
>
> I think I should modify both the svn trunk and 1.7 branch to:
>
> public void setCreateFile(boolean createFile) {
>          this.createFile = createFile;
> }
>
> Thoughts?
Yes,
  the original code is a bug.

Peter

>
> Kev
>
>
>  >>>
>
> Hi Kev,
>
> You don't know me; I subscribe to the various ant lists (ant-user,
> ant-dev, ant-cvs). I saw the below check-in and it just struck me as
> wrong (I don't think createFile should be set to the value of
> deleteOnExit); given that you're trying to get an alpha out, I thought
> I'd bring it to your attention. If it's correct, then please forgive the
> intrusion.
>
> Rick
>
> > From: kevj@apache.org [mailto:kevj@apache.org]
> > Sent: Sunday, November 11, 2007 6:41 PM
> > To: ant-cvs@apache.org
> > Subject: svn commit: r594009 -
> > /ant/core/branches/ANT_17_BRANCH/src/main/org/apache/tools/ant
> > /taskdefs/TempFile.java
>
> [snipped]
>
> > URL:
> >
> http://svn.apache.org/viewvc/ant/core/branches/ANT_17_BRANCH/src/main/or
> g/apache/tools/ant/taskdefs/TempFile.java?rev=594009&r1=594008&r2=594009
> &view=diff
> >
> ========================================================================
> ======
>
> [snipped]
>
> > @@ -123,6 +126,22 @@
> >      public boolean isDeleteOnExit() {
> >          return deleteOnExit;
> >      }
> > +
> > +    /**
> > +     * If set the file is actually created, if not just a
> > name is created.
> > +     * @param createFile boolean flag.
> > +     */
> > +    public void setCreateFile(boolean createFile) {
> > +        this.createFile = deleteOnExit;
> > +    }
> > +
> > +    /**
> > +     * Learn whether createFile flag is set for this tempfile task.
> > +     * @return the createFile flag.
> > +     */
> > +    public boolean isCreateFile() {
> > +        return createFile;
> > +    }
> >
> >      /**
> >       * Creates the temporary file.
> --
> Rick Genter
> Principal Software Engineer
> Silverlink Communications
> rgenter@silverlink.com
> www.silverlink.com
> Office (781) 425-5763
> Mobile (781) 771-9677
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org