Re: [picocontainer-dev] Re: the patch ?
Paul Hammant <[email protected]>
| Newsgroups | gmane.comp.java.picocontainer.devel |
|---|---|
| Message-ID | <[email protected]> |
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 ? - Paul On Nov 13, 2007, at 3:15 AM, Konstantin Priblouda wrote: > > --- Paul Hammant <[email protected]> wrote: > >> What do you think dude? >> >> Its the convergence of yours and mine >> >> Can I commit it ? ConfigParameter can be changed to >> use some of the >> functionality that it would commit... > > Yep. This is convergence - but I'm not very happy with > duplication of ocnverters (though there is nothing > that can be done about it) > > I think we also can shift conversion logic from > parameter to adapter ( thus eliminating need for > config parameter ) - we will need config adapter > then, > which would respond to inquires of all convertivble > types. Such config adapter (behaviour) would justify > addConfig() in interface, and also simplify scripting > stuff ( we are back to constant / reference routine - > no need to intruduce new xml elements ) > > WDYT? > > regards, > > ----[ Konstantin Pribluda http://www.pribluda.de ]---------------- > JTec quality components: http://www.pribluda.de/projects/ > > > > ______________________________________________________________________ > ______________ > Be a better sports nut! Let your teams follow you > with Yahoo Mobile. Try it now. http://mobile.yahoo.com/ > sports;_ylt=At9_qDKvtAbMuh1G1SQtBI7ntAcJ > > --------------------------------------------------------------------- > 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