Add RFC95 text: Use standard C/C++ integer types #8399
Conversation
rouault
force-pushed
the
rfc95_text
branch
3 times, most recently
from
fb614a7
to
23dc139
Compare
| 1) Proceed with them, keep GDAL 3.8 as version number. External code will have | ||
| to use #ifdef if support for GDAL < 3.8 and >= 3.8 is needed. | ||
|
|
||
| 2) Same as 1), but bump GDAL version number to 4.0. Cleaner way |
There was a problem hiding this comment.
I think doing this as a 4.0 change makes the most sense since it is a breaking change requiring changes in upstream codebases.
There was a problem hiding this comment.
@snowman2 @rouault I mentioned implications for Rasterio and Fiona on gdal-dev and will repeat myself here. I'm not sure what the necessary changes would be for them... they're Cython, not C, and the Cython project feels like conditional compilation directives was a mistake 😂
It's not clear to me that we can do just this https://github.com/PDAL/PDAL/pull/4179/files#diff-22b766759d40394aa74c390f2ff295b1737020f1a824e103238b60efa9e2814fR36 like for PDAL.
Introducing stubs to support type changes would be a drag, hopefully that could be avoided.
|
Merging as "proposed, but not adopted" |
Partially rendered view here