Looks fine; thank you.

Jody

On 02/06/2010, at 9:28 PM, Jody Garnett wrote:

> On 02/06/2010, at 7:16 PM, Michael Bedward wrote:
> 
>> Hi Jody,
>> 
>>> Updated the docs on this page:
>>> - http://docs.codehaus.org/display/GEOTDOC/08+FeatureCollection
>>> 
>>> To talk more about different feature collection options, and 
>>> SimpleFeatureCollection which is
>>> introduced on 2.7-M0.
>> 
>> Excellent - thanks for doing that.  I just had a look and have a
>> couple of questions...
>> 
>> The code snippet for ListFeatureCollection includes this:
>> 
>>  SimpleFeatureCollection collection = new ListFeatureCollection(type,list);
>> 
>> But the constructors are protected. Which is correct, the doc or the code ?
> 
> Neither! It is a work in progress :-( I must of written my test cases in the 
> same package; better change the code. Hopefully there is a public constructor 
> of some description.
> 
>> Do you want the page to refer to SimpleXXX classes throughout ? (except at 
>> the very beginning).
> 
> Yes I do - now that we have a 2.7-M0 out.
> 
> Jody


------------------------------------------------------------------------------

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

Reply via email to