Andrea Aime wrote:
> Gabriel Roldan ha scritto:
>> I know why it doesn't work, FastBBOX is tricky, it defeats the purpose
>> of DuplicatingFilterVisitor:
>>
>> public Object accept(FilterVisitor visitor, Object context) {
>> Object result = visitor.visit(this, context);
>> if(!(result instanceof BBOX))
>> return result;
>>
>> BBOX clone = (BBOX) result;
>> if(clone.getExpression1().equals(getExpression1()) &&
>> clone.getExpression2().equals(getExpression2()))
>> return new FastBBOX(property, envelope);
>>
>> return result;
>> }
>>
>> gonna see what else could be done
>
> Btw, I've suggested in IRC to make FastBBOX implement the deprecated
> interfaces if it's necessary.
> It's a package private class that is supposed to be a drop in
> replacement for a BBOX filter and well, it is for all modules that
> use the new filter interfaces, but not for the old ones.
> But we can make it implement the old ones as well
Understood, yet I would prefer a self contained hack on an unsupported
module that needs to be ported to the new filter interfaces rather than
polluting new stuff with the old ones. What do you think?
Gabriel
>
> Cheers
> Andrea
>
--
Gabriel Roldan
OpenGeo - http://opengeo.org
Expert service straight from the developers.
------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
