Resolved some of the review comments

Author: Kontinuation

c/sedona-gdal/Cargo.toml

@@ -29,9 +29,6 @@ readme.workspace = true
 edition.workspace = true
 rust-version.workspace = true
 
-[lib]
-crate-type = ["staticlib", "cdylib", "lib"]
-
 [dependencies]
 gdal-sys = { version = "0.12.0", optional = true }
 libloading = { workspace = true }

c/sedona-gdal/src/dyn_load.rs

@@ -79,7 +79,7 @@ macro_rules! load_fn_any {
     }};
 }
 
-/// Populate all 64 function-pointer fields of [`SedonaGdalApi`] from the given
+/// Populate all function-pointer fields of [`SedonaGdalApi`] from the given
 /// [`Library`] handle.
 fn load_all_symbols(lib: &Library, api: &mut SedonaGdalApi) -> Result<(), GdalInitLibraryError> {
     // --- Dataset ---

c/sedona-gdal/src/gdal_api.rs

@@ -30,9 +30,9 @@ use crate::gdal_dyn_bindgen::SedonaGdalApi;
 ///
 /// Panics if the function pointer is `None`. This is unreachable in correct usage
 /// because all function pointers are guaranteed to be `Some` after successful
-/// `sedona_gdal_dyn_api_init` (the C loader fails if any symbol is missing),
-/// and you cannot obtain a `&GdalApi` without successful initialization
-/// (it's behind `OnceLock`).
+/// initialization of [`GdalApi`] via [`GdalApi::try_from_shared_library`] or
+/// [`GdalApi::try_from_current_process`], and you cannot obtain a `&GdalApi`
+/// without successful initialization.
 macro_rules! call_gdal_api {
     ($api:expr, $func:ident $(, $arg:expr)*) => {
         if let Some(func) = $api.inner.$func {
@@ -80,7 +80,8 @@ impl GdalApi {
         &self.name
     }
 
-    /// Check the last CPL error and return a `GdalError` if there was one.
+    /// Check the last CPL error and return a `GdalError`, it always returns an error struct
+    /// (even when the error number is 0).
     pub fn last_cpl_err(&self, default_err_class: u32) -> GdalError {
         let err_no = unsafe { call_gdal_api!(self, CPLGetLastErrorNo) };
         let err_msg = unsafe {

c/sedona-gdal/src/gdal_dyn_bindgen.rs

@@ -66,7 +66,6 @@ pub enum GDALDataType {
     GDT_CFloat16 = 16,
     GDT_CFloat32 = 10,
     GDT_CFloat64 = 11,
-    GDT_TypeCount = 17,
 }
 
 impl GDALDataType {
@@ -90,7 +89,6 @@ impl GDALDataType {
             14 => Ok(GDALDataType::GDT_Int8),
             15 => Ok(GDALDataType::GDT_Float16),
             16 => Ok(GDALDataType::GDT_CFloat16),
-            17 => Ok(GDALDataType::GDT_TypeCount),
             _ => Err(()),
         }
     }
@@ -239,7 +237,7 @@ pub const wkbNDR: OGRwkbByteOrder = 1; // Little endian
 
 #[repr(C)]
 #[derive(Debug, Copy, Clone, Default)]
-pub struct SedonaGdalApi {
+pub(crate) struct SedonaGdalApi {
     // --- Dataset ---
     pub GDALOpenEx: Option<
         unsafe extern "C" fn(