Changes between Version 3 and Version 4 of rfc51_rasterio_resampling_progress

Timestamp:

Dec 1, 2014, 10:42:50 AM (9 years ago)

Author:

Even Rouault

Comment:

RFC 51: Make GDALRasterIOExtraArg* psExtraArg argument of RasterIO() optional for out-of-tree code

rfc51_rasterio_resampling_progress

@@ -122 +122 @@
 GDALDataset and GDALRasterBand (non virtual) RasterIO() and (virtual)
 IRasterIO() methods have a new final argument psExtraArg of type GDALRasterIOExtraArg*.
+This extra argument defaults to NULL for code using GDAL, but is required for
+all in-tree code, so as to avoid that in-tree code forgets to forwards psExtraArg
+it might have returned from a caller.
 
 GDALDataset::RasterIO() and GDALRasterBand::RasterIO() can accept a NULL pointer
@@ -197 +200 @@
 
   * C++ users of the GDALRasterBand::RasterIO() and
-    GDALDataset::RasterIO() API must add the new GDALRasterIOExtraArg* psExtraArg
-    argument (potentially to NULL). It would have been possible to declare it as
-    optional, but it would have then been more error prone to forget forwarding
-    psExtraArg. This was especially true in the conversion phase, but now it is
-    done, it isn't perhaps needed any more.
+    GDALDataset::RasterIO() API do not need to change their code, since the new
+    GDALRasterIOExtraArg* psExtraArg argument is optional for out-of-tree code.
 
   * Out-of-tree drivers that implement IRasterIO() must be changed to accept the new