You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by Jesse Glick <ty...@gmail.com> on 2013/03/31 13:45:42 UTC

Re: svn commit: r1462911 - in /ant/core/trunk/src: main/org/apache/tools/ant/taskdefs/Jar.java main/org/apache/tools/ant/taskdefs/Zip.java tests/junit/org/apache/tools/ant/taskdefs/ZipExtraFieldTest.java

On 03/31/2013 06:22 AM, bodewig@apache.org wrote:
> +    public static final class Zip64ModeAttribute extends EnumeratedAttribute {

BTW for new code you can simply use an enum.

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


Re: svn commit: r1462911 - in /ant/core/trunk/src: main/org/apache/tools/ant/taskdefs/Jar.java main/org/apache/tools/ant/taskdefs/Zip.java tests/junit/org/apache/tools/ant/taskdefs/ZipExtraFieldTest.java

Posted by Jesse Glick <jg...@cloudbees.com>.
On 03/31/2013 02:23 PM, Stefan Bodewig wrote:
> I must admit I haven't checked whether IntrospectionHelper supports
> enums, does it do so already?  If so, great.

Yes, see getEnumSetter.

> If I want a different string representation like "as-needed" then I
> still end up writing a mapping class

Or just use ‘as_needed’ or ‘asNeeded’ etc.


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


Re: svn commit: r1462911 - in /ant/core/trunk/src: main/org/apache/tools/ant/taskdefs/Jar.java main/org/apache/tools/ant/taskdefs/Zip.java tests/junit/org/apache/tools/ant/taskdefs/ZipExtraFieldTest.java

Posted by Stefan Bodewig <bo...@apache.org>.
On 2013-03-31, Jesse Glick wrote:

> On 03/31/2013 06:22 AM, bodewig@apache.org wrote:

>>+    public static final class Zip64ModeAttribute extends EnumeratedAttribute {

> BTW for new code you can simply use an enum.

I must admit I haven't checked whether IntrospectionHelper supports
enums, does it do so already?  If so, great.

If I want a different string representation like "as-needed" then I
still end up writing a mapping class and an enum wouldn't give me much
over EnumeratedAttribute.

Stefan

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