Ciao Bret, I had a quick look and I am happy withe changes. I will create a jira and backport to 9.x
Regards, Simone Giannecchini == Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information. == Ing. Simone Giannecchini @simogeo Founder/Director GeoSolutions S.A.S. Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 333 8128928 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- On Wed, Jul 17, 2013 at 3:28 AM, Brett Walker <brett.wal...@geometryit.com> wrote: > > > -----Original Message----- > From: Michael Bedward [mailto:michael.bedw...@gmail.com] > Sent: Wednesday, 17 July 2013 11:18 AM > To: Brett Walker > Cc: geotools-devel@lists.sourceforge.net; jody.garn...@gmail.com > Subject: Re: Pull Request to upgrade JAITools > > 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/35ee8f8434f2dbb435622 >> 671f725a08e295df848) >> >> 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
