GeneralEnvelope should accepts envelopes crossing the antimeridian
------------------------------------------------------------------

                 Key: GEOT-1724
                 URL: http://jira.codehaus.org/browse/GEOT-1724
             Project: GeoTools
          Issue Type: Improvement
          Components: core referencing
    Affects Versions: 2.5-M0
            Reporter: Martin Desruisseaux
            Assignee: Martin Desruisseaux
            Priority: Minor


{{GeneralEnvelope.setRange(int, double, double)}} currently contains this code:

{code:java}
if (minimum > maximum) {
    // Make an empty envelope (min == max)
    // while keeping it legal (min <= max).
    minimum = maximum = 0.5 * (minimum + maximum);
}
{code}

This approach has two problems:

* It is a cause of bug that may be long to identify, since the coordinates are 
silently modified. An exception, or at the very least a warning could save 
developper time.
* WCS authorizes (with special treatment) cases where {{maximum < minimum}} at 
least in the longitude case. They are envelope crossing the antimeridian.

We should accept {{maximum < minimum}} only for axis declared with 
{{RangeMeaning.WRAPAROUND}}. For other axis, we need to decide if we should 
throws an exception, reorder the minimum and maximum or just log a warning.

Every {{GeneralEnvelope}} methods like {{contains}} will need to be updated for 
taking the antimeridian crossing case in account.


-- 
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: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to