Added Rasterband::set_no_data accepting Option<f64>.

CHANGES.md

@@ -10,6 +10,11 @@
 
   - <https://github.com/georust/gdal/pull/303>
 
+- Added ability to delete no-data when `None` is passed to `RasterBand::set_no_data(&mut self, no_data: Option<f64>))`
+
+  - <https://github.com/georust/gdal/pull/308>
+
+
 ## 0.13
 
 - Add prebuild bindings for GDAL 3.5

src/raster/rasterband.rs

@@ -431,12 +431,27 @@ impl<'a> RasterBand<'a> {
     }
 
     /// Set the no data value of this band.
+    ///
+    /// See [`set_no_data`](Self::set_no_data) for similar function
+    /// where _no-data_ can also be deleted.
     pub fn set_no_data_value(&mut self, no_data: f64) -> Result<()> {
-        let rv = unsafe { gdal_sys::GDALSetRasterNoDataValue(self.c_rasterband, no_data) };
+        self.set_no_data(Some(no_data))
+    }
+
+    /// Set the no data state of this band.
+    /// If `no_data` is `None`, any existing no-data value is deleted.
+    pub fn set_no_data(&mut self, no_data: Option<f64>) -> Result<()> {
+        let rv = if let Some(no_data) = no_data {
+            unsafe { gdal_sys::GDALSetRasterNoDataValue(self.c_rasterband, no_data) }
+        } else {
+            unsafe { gdal_sys::GDALDeleteRasterNoDataValue(self.c_rasterband) }
+        };
+
         if rv != CPLErr::CE_None {
-            return Err(_last_cpl_err(rv));
+            Err(_last_cpl_err(rv))
+        } else {
+            Ok(())
         }
-        Ok(())
     }
 
     /// Returns the color interpretation of this band.

src/raster/tests.rs

@@ -559,6 +559,8 @@ fn test_set_no_data_value() {
     assert_eq!(rasterband.no_data_value(), None);
     assert!(rasterband.set_no_data_value(1.23).is_ok());
     assert_eq!(rasterband.no_data_value(), Some(1.23));
+    assert!(rasterband.set_no_data(None).is_ok());
+    assert_eq!(rasterband.no_data_value(), None);
 }
 
 #[test]