(Jody-I am still unable to post to geotools-devel@lists.sourceforge.net, although I am subscribed and receive mail from that list. Could you forward this to the list? Thanks!)
I have been looking into GEOT-411, listed under the "Trivial" bugs category. I have sketched out some modifications to MolodenskiTransform.java that would take care of the issues involving overwriting the array of source points when the dimensions are not equal (2 or 3 dimensions are allowed for either source or target). Basically the issue is that, depending on the starting indices, it can be a little tricky to avoid overwriting a point that hasn't been transformed yet (when the target array is the same as the source array). At present there is only a single "while" loop with the option to run either forward or backwards over the indices. This is sufficient to handle the cases when the dimensions are equal, but not when the dimensions are different. My solution to this would be to split the source array into the parts that can be safely overwritten by iterating in the forward direction, and the part that can be safely overwritten by iterating in the backwards direction, and run a while loop over these two parts separately. In some cases (such as no overwriting, or overwriting with equal dimensions) the condition to enter one of the while loops would never be satisfied, and only one would run. I would like to test this out to make sure it will really work OK. However I don't see any existing test code for this class - is this correct, or am I missing something? This is especially significant because I think I have found an error in the original code that could cause an infinite loop, and I would like to verify and correct that before moving on to modifications. Thanks, Tara ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
