ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Bodewig <bode...@apache.org>
Subject Re: Task getTaskType() and setTaskType()
Date Fri, 26 Sep 2003 07:46:26 GMT
On Thu, 25 Sep 2003, Larry Shatzer <Larry.Shatzer@spirent.com> wrote:

> Is there any reason why getTaskType() is protected?

The better question would be, is there any reason why setTaskType
isn't package private anymore.

task-type is supposed to be the name that has been used in the build
file.  task-name is generally the same but can be reset by the user
using the taskname attribute shared by all tasks.

task-type used to be package private to ensure that user's can't
fiddle with it.  Expand, the implementation of <unzip> uses this value
to print out a deprecation message even if the user does something
silly like

        <expand taskname="unzip" .../>

Costin changed it to public more than a year ago, the comment says

,----
| Allow the task type to be set from a differnet package. That elminate
| the need to have RuntimeConfigurable2 in the same package.
`----

Given that there is no RuntimeConfigurable2 anymore, I'm not sure
whether it really still needs to be public at all.  If it remains
public, the getter should become public as well, I agree.

Stefan

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


Mime
View raw message