Re: Expanding Environmental variables patch

2008-05-14 Thread Emmanuel Bourg
Hi there, I'm a developer at Apache Commons and Charles contacted us about its issue with ExtendedProperties. ExtendedProperties is no longer evolving in Commons Collections since it forked to spawn the Commons Configuration project some years ago. Many features have been implemented in

Re: Expanding Environmental variables patch

2008-05-07 Thread Christopher Schultz
Charles, Sanders, Charles wrote: I actually dislike modifying passed-in parameters Why? The documentation for RuntimeInstance.setConfiguration does not state that the parameter is modified, so it's rude to modify it. Don't forget that the documentation is part of the API, too, and it's

Re: Expanding Environmental variables patch

2008-05-07 Thread Will Glass-Husain
: Expanding Environmental variables patch Charles, Sanders, Charles wrote: I actually dislike modifying passed-in parameters Why? The documentation for RuntimeInstance.setConfiguration does not state that the parameter is modified, so it's rude to modify it. Don't forget

Re: Expanding Environmental variables patch

2008-05-06 Thread Christopher Schultz
Charlei, csanders wrote: Attached is the patch, adding three new functions, expandEnvironmentalVariables( ExtendedProperties ), expandEnvironmentalVariables(Vector ), Any reason not to accept any Collection rather than limiting this to just a Vector? Forgive me if there are API