Add `Rasterband::fill` - Githubissues

netthier commented 2 months ago

[x] I agree to follow the project's code of conduct.
[x] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

This PR adds safe bindings for https://docs.rs/gdal-sys/latest/gdal_sys/fn.GDALFillRaster.html (https://gdal.org/api/gdalrasterband_cpp.html#classGDALRasterBand_1a55bf20527df638dc48bf25e2ff26f353) The C++ API has a default value for the dfImaginaryValue param which is missing from the C API, so I reintroduced via an Option+.unwrap_or. Let me know if this isn't acceptable. I also haven't tested that parameter, as I'm unsure how to read it back.

netthier commented 2 months ago

I should probably add docs about the default imaginary value too...

netthier commented 2 months ago

I ran cargo fmt against it, CI shouldn't fail anymore (I hope).

georust / gdal