HI Jim, > All of those conditions are optimized equivalents to getting the > determinant. > > invert() is a destructive call, but since getTransform is documented to > return a copy, that is fine - but getDeterminant() will involve less math to > check the condition...
Thanks for the clarification. Please find the latest version of the patch at http://93.83.133.214/8162591/webrev.04/ - where catching the Exception has been replaced with a != 0.0 check against the determinant. Please commit in case no further issues are found. Best regards, Clemens