Agree to remove the CoreDeploymentInfo interface, we only have one implementation, and the codes always need to do the some extra cast actions.
2010/9/3 David Blevins <[email protected]> > Since the little refactoring we did to add AppContext, ModuleContext and > MethodContext I've been dying to rename CoreDeploymentInfo to a simpler > BeanContext. So that would give us a cleaner: > > AppContext > \-- ModuleContext > \--BeanContext > \--MethodContext > > At the same time we would just remove the DeploymentInfo interface which is > an abstraction that never panned out and this point is just confusing why > it's there. It's also bothered me that CoreDeploymentInfo has the 'Info' > suffix which confuses it with the InfoObject tree. > > Didn't want to do this till the merge was done, but now we're done with > that. Still need to fix our build issues, but would like to do the quick > rename after that. Throwing it out now to give time for input. We might > want to apply any outstanding patches before doing the rename. > > Thoughts? > > -David > > -- Ivan
