Thanks for the tip; the patch also contained some modifcations to demo Csv2Shp
Jody

On 05/08/2010, at 6:41 PM, Andrea Aime wrote:

> Jody Garnett ha scritto:
>> Sounds fine andrea; if possible I would like to use the data module
>> (for something!) or kill it. I figure it is the same purpose as the
>> jdbc module (helper classes, root implementations and so forth).
>> At least for the first cut do what we did with AbstractDatastore and
>> rely on the test cases in shapefile datastore to cover the base class.
>> You may want to teach the shapefile factory to do the actual creation
>> (so you can reuse the param definition). It already has the task of
>> select the best implementation based on parameters so it is a good
>> fit. I know it is a bit rude making the factory smart (ie have logic)
>> but it does beat people expecting to call DataStoreFinder and should
>> be less work for you.
> 
> Here is a jira and a patch for your reviewing pleasure:
> http://jira.codehaus.org/browse/GEOT-3233
> 
> (suggestion, the patch is big because the directory module
> contained a lot of dead gml data, just look for the bits
> in modules/library/data, modules/plugin/shapefile)
> 
> Cheers
> Andrea
> 
> 
> -- 
> Andrea Aime
> OpenGeo - http://opengeo.org
> Expert service straight from the developers.


------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share
of $1 Million in cash or HP Products. Visit us here for more details:
http://p.sf.net/sfu/dev2dev-palm
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to