<html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8"></head><body ><div>Dmitriy, </div><div><br></div><div>The main reason for the new symbol is that if we change the return type it might cause crashing problems that are not detected at compile time if GetFID is used in a printf like function. For example</div><div><br></div><div>printf("%ld %s", GetFID (), str)</div><div><br></div><div>Even</div><div><br></div>---<div>Spatialys - Geospatial professional services</div><div>http://www.spatialys.com</div><br><br><div>-------- Message d'origine --------</div><div>De : Dmitriy Baryshnikov <bishop.dev@gmail.com> </div><div>Date :24/01/2015 14:07 (GMT+01:00) </div><div>A : gdal-dev@lists.osgeo.org </div><div>Objet : Re: [gdal-dev] Call for discusson on RFC 31 - OGR 64bit Integer
Fields and FIDs </div><div><br></div>Hi Even,<br><br>The RFC looks good. One note. May be in GDAL 2 we can use GetFID instead <br>GetFID64? Because we can allow some incompatibility between GDAL 1.x and <br>GDAL 2.x.<br><br>Best regards,<br> Dmitry<br><br>23.01.2015 18:17, Even Rouault пишет:<br>> Hi,<br>><br>> This is a call for discussion on revisiting the existing RFC 31 - OGR 64bit<br>> Integer Fields and FIDs, initiated by Frank in 2010 and that I've extended.<br>><br>> http://trac.osgeo.org/gdal/wiki/rfc31_ogr_64<br>><br>> Below the summary :<br>> """<br>> This RFC addresses steps to upgrade OGR to support 64bit integer fields and<br>> feature ids. Many feature data formats support wide integers, and the<br>> inability to transform these through OGR causes increasing numbers of<br>> problems.<br>> """<br>><br>> A first discussion took place at http://lists.osgeo.org/pipermail/gdal-<br>> dev/2010-November/thread.html#26883. My main remarks at that time were about<br>> compatibility issues, but as this would go in GDAL 2.0, they are likely less<br>> critical.<br>><br>> Even<br>><br><br>_______________________________________________<br>gdal-dev mailing list<br>gdal-dev@lists.osgeo.org<br>http://lists.osgeo.org/mailman/listinfo/gdal-dev</body>