Re: [picocontainer-dev] Re: the patch ?
Konstantin Priblouda <kpriblouda-/[email protected]>
| Newsgroups | gmane.comp.java.picocontainer.devel |
|---|---|
| Message-ID | <[email protected]> |
--- Paul Hammant <[email protected]> wrote: > Committed. > > Next up is most likely - > > 1) move NoConverterAvailableException to core and > ensure that > BasicComponentParameter is throwing it. > 2) snip away at some of the bits of ConfigParameter > (as they are > duplicated in the parent class) > 3) put in more converters (and test cover) > 4) review again Argumentative, Properties and System > properties > PicoContainers. > > 5) move conversion logic to some better home (as you > suggest) and > BasicComponentParameter is not correct for it long > term > > Wanna give it a go ? Well, looking at your changes it seems that: - no need for ConfigParameter anymore. - no need for special adapter anymore ( because conversion logic is in basic component adapter now ) Well, in this case basic component adapter also becomes obsolete... So, we can just merge all the conversion stuff into component parameter, and enooy new functionality. regards, ----[ Konstantin Pribluda http://www.pribluda.de ]---------------- JTec quality components: http://www.pribluda.de/projects/ ____________________________________________________________________________________ Get easy, one-click access to your favorites. Make Yahoo! your homepage. http://www.yahoo.com/r/hs --------------------------------------------------------------------- To unsubscribe from this list please visit: http://xircles.codehaus.org/manage_email