Summary: | avoid parallel arrays of base types | ||
---|---|---|---|
Product: | Tomcat 7 | Reporter: | Ralf Hauser <hauser> |
Component: | Catalina | Assignee: | Tomcat Developers Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | ||
Priority: | P4 | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All | ||
Attachments: | patch_parallelArrays.txt |
Description
Ralf Hauser
2010-03-08 06:55:57 UTC
Ralf, Why did you submit a patch with deprecated items in it? Instead of documenting the items to be deprecated, just go all the way with the enum. Also, I'm not sure that BootLocation necessarily needs to implement CharSequence. I think it would be cleaner for BootLocation to simply contain a String (location) and enum type. I don't happen to like the name BootLocation but that's just a matter of taste. Maybe RepositoryLocation? Fixed in trunk and will be in 7.0.9 onwards. Since I agree with Chris's views, the patch applied was along those lines. |