Andrea Aime wrote:
>> The duplication of method names results in hard to understand code. 
>> Here is the suggested improvement:
>> - FeatureType.getAttributeDescriptor(): AttributeDescriptor
>> - Feature.getAttribute(): Attribute
> Yes, I like the proposed solution better. I believe the original 
> method names were shortened to save space.
I just had a go at changing this; but then it was hard to stop; ie we 
should have getAttributeDescriptors() and getPropertyDescriptors() and 
so on ... so I backed out my change. I need some kind of "heck yeah" 
from the usual crowd of people who care about this stuff?

Acuster ping? what are your thoughts? Is this worth the trouble now? 
Will it save us enough trouble later? I think Landon was also interested 
in this inconsistency?

Jody
PS. Andrea it will be better to load GeoAPI and GeoTools and GeoServer 
into the same workspace and use "refactor" to accomplish this rename.

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to