Justin Deoliveira ha scritto:
> Nice patch! Big but very clean.
> 
> The only minor suggestion/note i have is that 
> WMSLayerINfoImpl.getWMSLayer() does a lot of work inline. The other 
> ResourceInfoImpl classes delegate out to ResourcePool for all resource 
> loading operations. It would be nice to follow that same pattern. 
> Especially if the proposal to make the catalog classes pure java beans 
> ever comes to light.

Oh, ok. I thought ResourcePool was to be involved only when pooling
resoures was required.
But sure, an easy-peasy change

Cheers
Andrea


-- 
Andrea Aime
OpenGeo - http://opengeo.org
Expert service straight from the developers.

------------------------------------------------------------------------------
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to