>Jesse Glick wrote: >> [EMAIL PROTECTED] wrote: >>>> It seems safe and useful for 1.7 . >>> >>> +1
useful: +1, many people are writing on Java5, Java6 is ante portas. >> Any other +/-? It's not really clear to me what, if any, criteria >> there are for RFEs this late in the cycle. I *think* the patch is safe >> but you never know. Yes - very late. It's not a checkstyle or doco-patch ... You have done a complete test I think? >the only thing I see this breaking is any class which has a >java5 enum of a specific name, and a legacy enum with the same >name. Is this likely? Not yet. You mean that? public class T extends org.apache.tools.ant.Task { public enum ThingEn { first, second, third } private ThingEn enThing; public void setThing(ThingEn e) { thing = e; } public ThingEA extends EnumeratedAttribute { public String[] getValues() { return new String[]{"first", "second", "third"}; } } private ThingEA eaThing; public void setThing(ThingEA e) { eaThing = e; } } --> two "setThing()" methods >Oh, and you end up writing tasks that are java5+ only, of course. Nothing for Ant itself - but if the writer wants to use Java5, he will do that. But supporting enum it would be easier to forget, that enum is Java5+ ... that's right. Maybe we could spent a refactoring "convert enum to EnumeratedAttribute" ;-) >If the change does go in, I'd have to update that bit of Ant >in Action (that's its new name, BTW), which is not too late. hihi .... "Author in Action" ;-) Jan --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]