On Fri, Jan 28, 2011 at 12:59 PM, Jody Garnett <[email protected]> wrote:
> Hi Andrea; I just lurked last during the last conversation. The idea is a 
> sensible one; a couple questions. Do we need to be able to check this out 
> when making a function (Is it information that should be available as part of 
> FunctionFactory interface; or as part of FunctionName?)

As I suggested in my previous mail, it would be a interface the
function implements.
So the filter simplifier would do
if(!(function instanceof DynamicFunction)) {
...
}

to guard the simplification procedure.

> Or are you content to just have the marker interface VolitileFunction so you 
> can check the result at runtime. I also note that marker interfaces have 
> fallen a bit out of use in favour of annotations. Not sure we would care.

That works for me because the simplification happens against a already
created filter,
which has the function interfaces in place already.

I'd go for the marker interface, but it's just habit, having a @VolatileFunction
would work just as good I guess (err... or maybe not... are
annotations preserved
when one uses dynamic proxies and the like?)

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
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to