Hi Brett, Thanks for this work.
Yes, there were substantial changes to RangeLookupTable and related classes to fix thread safety, so it's no surprise that you had to hack things at the GeoTools end. However, I don't know the CoverageUtilities code at all - it would be best to ask Simone, Andrea or Daniele to review it. Michael On 16 July 2013 22:21, Brett Walker <brett.wal...@geometryit.com> wrote: > Hi, > > I have made a pull request to upgrade JAITools to version 1.3.1. > > To get the build to pass it seems that I had to brutalise a method in > CoverageUtilities.java > (https://github.com/brett-walker/geotools/commit/35ee8f8434f2dbb435622671f725a08e295df848) > > A question for Jody or Michael. > > Do the changes look right? Have I changed the method's meaning too > drastically? Or is the change from version 1.3.0 to 1.3.1 to drastic for > CoverageUtilities do be altered that the upgrade is not possible? > > The process-raster module, of which the CoverageUtilities is apart, is in > the unsupported part of GeoTools. > > Brett ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
