>>> I would like to see the "geometry" methods be consistent with the >>> getAttribute()/getProperty()" >>> - getAttribute( Name ): Object >>> - getGeometryAttribute( Name ): Object (depends on your Geometry >>> implementation) >>> - getGeometryProperty( Name ): GeometryAttribute >>> - getProperty( Name ): Property >>> I still think that leaving getGeometryAttribute as getDefaultGeometry is a big win for the transition. But in my mind making it easy to transition to is a bigger win then consistent naming conventions.
But as I said we are at an impasse on this one, we will Andrea to be the tie breaker. >> Can you include return types of the methods please :) >> > I fixed the above - and see a problem: getGeometryProperty( Name ) > returns a GeometryAttribute? Do we care? >>> Fun ;-) Justin you better put a deadline on the second round of review. >>> >> Ideally we can get everyones concerns addressed and out in the open by >> Friday... but that may be pushing it. >> > Justin we need to push it - how about we start working against these > things on Friday, and we close the door Monday? I doubt we will get a > good review until people can run sample code anyways? > > Jody > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2005. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > Geotools-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/geotools-devel > > !DSPAM:4007,46e990d5185055332866982! > -- Justin Deoliveira The Open Planning Project http://topp.openplans.org ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
