S. Forbes wrote:
> Howdy,
>
Hi again:
I picked this one up as well. We best make a Jira issue and talk to the
module maintainer :-)
Cory Horner - meet Shaun - apparently Shaun is doing a code review and
turning up trouble; in a surprise move he is also
providing fixes.
Jody
> It appears to me as though a 'break' could be included when checking
> logic filters once an unsupported filter has been found to stop
> redundant checks from being performed.
>
> See:
> org.geotools.filter.FilterCapabilities
> boolean fullySupports(Filter filter)
>
> From:
> while (filters.hasNext()) {
> testFilter = (Filter) filters.next();
>
> if (!(this.fullySupports(testFilter))) {
> supports = false;
> }
> }
>
> To:
> while (filters.hasNext()) {
> testFilter = (Filter) filters.next();
>
> if (!(this.fullySupports(testFilter))) {
> supports = false;
> break;
> }
> }
>
> Cheers,
> Shaun
>
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys-and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Geotools-gt2-users mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
>
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Geotools-gt2-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
