Well unfortunately the old notion of a "type" does not lign up with the new notion of a "type", it lines up with "descriptor" in the new model. But yeah I agree, makes it hard.
Still wondering why you are worrying about this if you are strictly documenting against the geoapi interfaces. Jody Garnett wrote: > I just noticed the duplication is all. > > I am also finding a few more things, > org.geotools.feature.GeometryAttributeType does not implement the > org.opengis interface of the same name and so on. Makes it tough when I > am trying to use this api ... in particular: > > type.find( type.getDefaultGeometry() ); > > Suppose this is why it is a release candidate ;-) > Jody > > > Justin Deoliveira wrote: >> A bit late to come up with a objection to the name as I beat these down >> peoples throats with multiple email threads before :). But I >> understand... you are only now getting a chance to do a formal review. >> >> So I am little confused. If your docs are constrained to SimpleFeature >> then why does the geotools.Feature interface matter to you? >> >> Jody Garnett wrote: >> >>> Hi Justin; >>> >>> I am running into some inconsistences as I document up examples that >>> are restricted to SimpleFeature and so forth... >>> >>> org.geotools.feature.Feature.getPrimaryGeometry(): Geometry >>> - a geometry that is somehow "preferred" and used as the default when >>> rendering. In many cases this is simply the first geometry declaired. >>> >>> The same concept is represented in super classes ... >>> >>> org.opengis.feature.Feature.getDefaultGeometry(): GeometryAttribute() >>> org.opengis.feature.simple.SimpleFeature.getDefaultGeometryValue(): >>> Object >>> - helper method for getDefaultGeometry().getValue() >>> - may be type narrowed by an implementation >>> >>> I would like to remove getPrimaryGeometry(): Geometry and replace it >>> with getDefaultGeometryValue() type narrowed to Geometry for two >>> reasons ... >>> - code completion will let people discouver the new method when they >>> start typing getDefaultGeometry >>> - on less method for people to figure out >>> >>> What do you think? >>> Jody >>> >>> ------------------------------------------------------------------------- >>> >>> This SF.net email is sponsored by: Splunk Inc. >>> Still grepping through log files to find problems? Stop. >>> Now Search log events and configuration files using AJAX and a browser. >>> Download your FREE copy of Splunk now >> http://get.splunk.com/ >>> _______________________________________________ >>> Geotools-devel mailing list >>> [email protected] >>> https://lists.sourceforge.net/lists/listinfo/geotools-devel >>> >>> >>> >>> >> >> >> > > > !DSPAM:4007,46c92d57110005210051143! > -- Justin Deoliveira The Open Planning Project http://topp.openplans.org ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
