[
http://jira.codehaus.org/browse/GEOT-746?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Martin Desruisseaux reopened GEOT-746:
--------------------------------------
The commited Range implementation has some problems. The implementation of
"contains(Range)" method seems to do the work of a "intersects(Range)" method
(maybe from a copy-and-paste with the edition step forgoten). The "compareMax"
method has the sign reverted in the sequence of "if" statements in the "compare
== 0" case. Or actually the later problem hide a more subtle problem. The
result of both "compareMin" and "compareMax" depends on whatever the given
"value" argument is a minimum or a maximum. If a minimum, the first inclusive
value is after that minimum. If maximum, the first inclusive value if before
that maximum. A boolean "isInclusive" argument is not enough - we need to
consider in which direction is the inclusive value.
I'm trying to fix those issues...
> Remove JAI dependency from the main module
> ------------------------------------------
>
> Key: GEOT-746
> URL: http://jira.codehaus.org/browse/GEOT-746
> Project: GeoTools
> Issue Type: Task
> Components: core main, core referencing
> Affects Versions: 2.2.1, 2.3.0, 2.4-M0
> Reporter: Martin Desruisseaux
> Assignee: Martin Desruisseaux
> Priority: Minor
> Fix For: 2.5-M1
>
>
> Except for the GridCoverage implementation, the JAI dependency in the main
> module is very small. The only JAI class used is:
> javax.media.jai.util.Range
> It would be easy to provide our own Range implementation in the
> org.geotools.util package. The proposed transition path is:
> - Create an org.geotools.util.Range class which extends
> javax.media.jai.util.Range for now (for a smooth transition).
> - Put a clear warning saying that the JAI Range inheritance will be
> removed in a future release.
> - Wait for users to replace "import javax.media.jai.util.Range" by "import
> org.geotools.util.Range". No more change required.
> - After the 2.2 release, remove the "extends javax.media.jai.util.Range"
> clause in the org.geotools.util.Range class.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
