Re: [picocontainer-dev] Re: the patch ?
Paul Hammant <[email protected]>
| Newsgroups | gmane.comp.java.picocontainer.devel |
|---|---|
| Message-ID | <[email protected]> |
There's still some functionality that ConfigParameter has that BasicParameter does not, so I'll merge that in before deleting (unless you want to do it Konstantin), You've made quite a powerful feature that will be great in the core, even it the type-safety police have heart attacks.. - Paul On Nov 14, 2007, at 1:27 AM, Konstantin Priblouda wrote: > > --- 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 > --------------------------------------------------------------------- To unsubscribe from this list please visit: http://xircles.codehaus.org/manage_email