Hi, looking at the patches on ParameterParser I saw the latest patch changed the add(String, String) to map directly to append(String, String) insrtead of leaving it orthogonal to the add(String, int/long/double/Integer) methods - were it defines a new entry (removing anything old). <proposal> To avoid understanding problems, it might be necessary to change the method names from 'add' to 'set', since add is uderstood as 'append'. The we would have nice overloaded set(key, XXX), and all of the getXXX(key[, default]) methods, plus the append(key, String) one. The append method could be overloaded for completeness, but since it has not been a requirement up to now, it could wait till it itches someone :) Maybe the 'append' should be completely replaced by 'add', thus having orthogonal get/add/set methods, and being method level compatible to the current version, unless someone relies on add doing a replace - which could probably break a lot of code. Any objections/suggestions? If not I'll submit a patch in the next days (changing the 'add' method names to 'set') :) Christoph ------------------------------------------------------------ To subscribe: [EMAIL PROTECTED] To unsubscribe: [EMAIL PROTECTED] Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/> Problems?: [EMAIL PROTECTED]
