Re: [Geotools-devel] Literal guessing its type: a set of two patches to end this

[Jody G. <jod...@gm...>]

Sounds good; did the change have the expected performance gain?

Jody

On 30/01/2011, at 1:45 AM, Andrea Aime wrote:

> Hi,
> so I've been trying to remove the type guessing code in
> LiteralExpressionImpl.evaluate(object) and
> came up with two little patches, one for gt2, one for geoserver, that
> put together make them
> both build.
> 
> The gt one gets rids of the guessing game, fixed the only direct test
> against that game to use
> the context class param, and another test that failed to properly
> provide the evaluation context.
> 
> The gs one fixes the kml encoder to provide the context when
> evaluation expression, many
> calls were still missing it.
> 
> All in all no big changes.
> I've attached the patches here: http://jira.codehaus.org/browse/GEOT-3399
> 
> I'm considering committing to trunk once 2.7-RC1 is done and branched off.
> Opinions?
> 
> Cheers
> Andrea
> 
> -- 
> Ing. Andrea Aime
> Technical Lead
> 
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054  Massarosa (LU)
> Italy
> 
> phone: +39 0584962313
> fax:     +39 0584962313
> 
> http://www.geo-solutions.it
> http://geo-solutions.blogspot.com/
> http://www.linkedin.com/in/andreaaime
> http://twitter.com/geowolf
> 
> -----------------------------------------------------
> 
> ------------------------------------------------------------------------------
> Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
> Finally, a world-class log management solution at an even better price-free!
> Download using promo code Free_Logger_4_Dev2Dev. Offer expires 
> February 28th, so secure your free ArcSight Logger TODAY! 
> http://p.sf.net/sfu/arcsight-sfd2d
> _______________________________________________
> Geotools-devel mailing list
> Geo...@li...
> https://lists.sourceforge.net/lists/listinfo/geotools-devel