Ciao Ivan, well, what you are suggesting (deprecation/removal) cycle is a standard policy within the porjects we have been involved, therefore I am in favour of it. It is a natural way for having an API evolving.
On the other hand GDAL is well known for the rock solid stability of its API, but yeah this "instability" would apply only to the Java bindings and would follow a well known practice. Simone. ------------------------------------------------------- Ing. Simone Giannecchini GeoSolutions S.A.S. Founder Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584962313 fax: +39 0584962313 mob: +39 333 8128928 http://www.geo-solutions.it http://geo-solutions.blogspot.com/ http://www.linkedin.com/in/simonegiannecchini http://twitter.com/simogeo ------------------------------------------------------- On Fri, Dec 10, 2010 at 3:22 PM, Ivan Lucena <ivan.luc...@pmldnet.com> wrote: > Ciao Simone, Salut Even, > > You guy did a great job improving the performance of reading and writing > pixels buffer. > > http://www.mail-archive.com/search?l=gdal-dev%40lists.osgeo.org&q=ReadRaster_Direct > > But, "With great power comes great responsibility." > > Now we find users asking for changes that are not essential and that break > backward compatibility. > > Let's see this one here for example: > >> Method names begin with Capital letters. The Java standard is that >> methods always begin with lowercase letters; classes begin with Upper Case >> letters. Constants should be all UPPER_CASE. >> >> Dataset hDataset = >> gdal.Open(filename,gdalconstConstants.GA_ReadOnly).GetDriver().getLongName(); >> >> should read >> >> Dataset hDataset = >> Gdal.open(filename,GdalconstConstants.GA_READONLY).getDriver().getLongName(); > > So what should we do? > > Prepare for change in GDAL 2.0 sound good to me. > > We can have two Java wrapper with a deprecated warning on the older (maintain > it for about a year or two) and the new one would complain with Java > standards and updated to whatever the language have evolved so far. > > IMHO. > > Regards, > > Ivan > > >> -------Original Message------- >> From: Simone Giannecchini <simone.giannecch...@geo-solutions.it> >> To: Even Rouault <even.roua...@mines-paris.org> >> Cc: gdal-dev@lists.osgeo.org, Viji V Nair <v...@fedoraproject.org> >> Subject: Re: [gdal-dev] image-io extension >> Sent: Dec 10 '10 04:27 >> >> Ciao Even, >> I have asked daniele to make those changes (which have not been reviewed >> yet) >> in order to move away from the explicit usage of Vector and HashTable in >> method >> signatures in favour of using the respective interfaces (List and Map). >> Daniele can add more as needed... >> >> As you mentioned this change can break old code, but it should >> make the java bindings more ... javaish :). >> >> We have other changes in the queue which we might submit to your review >> early next year, but this is another topic... >> >> >> Viji , I don't think you need to use the patch you are referring to since >> that is needed for imageio-ext 1.1 which has not been released yet. >> The latest (and greatest) is 1.0.8 which uses an older gdal version. >> >> Ciao, >> Simone. >> ------------------------------------------------------- >> Ing. Simone Giannecchini >> GeoSolutions S.A.S. >> Founder >> >> Via Poggio alle Viti 1187 >> 55054 Massarosa (LU) >> Italy >> >> phone: +39 0584962313 >> fax: +39 0584962313 >> mob: +39 333 8128928 >> >> >> http://www.geo-solutions.it >> http://geo-solutions.blogspot.com/ >> http://www.linkedin.com/in/simonegiannecchini >> http://twitter.com/simogeo >> >> ------------------------------------------------------- >> >> >> >> On Thu, Dec 9, 2010 at 8:25 PM, Even Rouault >> <even.roua...@mines-paris.org> wrote: >> > Hi Viji, >> > >> > I maintain the GDAL Java bindings, but the image-io GDAL plugin is another >> > project whose maintener is Daniele I think. He'll correct me if I'm >> wrong, but >> > I saw his name on >> http://docs.codehaus.org/display/GEOTOOLS/ImageIO-EXT+GDAL >> > ;-) . I've CC'ed him so he has a chance to comment and explain the >> rational of >> > the patch. >> > >> > I wasn't aware of this patch. I've skimmed through it quickly, and it >> seems >> > that it only replaces old-fashionned data structures like Vector to use >> more >> > trendish ArrayList + Java 1.5 generics. Well, this is certainly not a bad >> idea >> > (something to consider for GDAL 2.0 ?), but I fail to see why it was >> > *necessary* . I'm not particularly keen on applying this patch in GDAL >> > mainline as it changes the API of the Java bindings, thus breaking working >> > code of other users. I fail to see why ImageIO-EXT cannot use the existing >> > API, even if it is perhaps not as convenient as it could be if it was >> using >> > more modern Java API. >> > >> > Best regards, >> > >> > Even >> > >> > Le jeudi 09 décembre 2010 14:18:18, Viji V Nair a écrit : >> >> Hi, >> >> >> >> I am one of the GDAL co-maintainer for Fedora, I would like to have >> >> the image-io extension support, currently it been supported through >> >> the following patch from image-io. But I dont think this is a good >> >> idea as the modifications are bit heavy. >> >> >> >> >> https://imageio-ext.dev.java.net/svn/imageio-ext/trunk/patches/gdal1.7.2.pa >> >> tch (Username : guest, Password: guest) >> >> >> >> Is there any work going on related to this? Please share the ideas >> >> >> >> Thanks >> >> Viji >> >> _______________________________________________ >> >> gdal-dev mailing list >> >> gdal-dev@lists.osgeo.org >> >> http://lists.osgeo.org/mailman/listinfo/gdal-dev >> > _______________________________________________ >> > gdal-dev mailing list >> > gdal-dev@lists.osgeo.org >> > http://lists.osgeo.org/mailman/listinfo/gdal-dev >> > >> _______________________________________________ >> gdal-dev mailing list >> gdal-...@lists.osgeo.org >> http://lists.osgeo.org/mailman/listinfo/gdal-dev >> > _______________________________________________ > gdal-dev mailing list > gdal-dev@lists.osgeo.org > http://lists.osgeo.org/mailman/listinfo/gdal-dev > _______________________________________________ gdal-dev mailing list gdal-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/gdal-dev
