You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tomcat.apache.org by bu...@apache.org on 2010/11/09 23:45:10 UTC

DO NOT REPLY [Bug 48870] avoid parallel arrays of base types

https://issues.apache.org/bugzilla/show_bug.cgi?id=48870

--- Comment #2 from Christopher Schultz <ch...@christopherschultz.net> 2010-11-09 17:45:05 EST ---
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?

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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