Maintenance cleanup on `dataset.rs`

metasim commented 9 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.

Changes:

Moved ancillary types from dataset.rs into their own or related files.
Moved non-core Dataset methods into respective raster or vector modules. Aims to simplify maintainability via "separation of concerns".
Removed unsafe from pointer accessor methods.

Possibly breaking changes:

Moved LayerIterator, LayerOptions and Transaction to crate::vector.

Aside: The impetus for this refactor is to focus the changes necessary for adding Dataset to https://github.com/georust/gdal/pull/445. But even if that PR takes a different route, this change will benefit long-term maintenance.

metasim commented 9 months ago

bors r=lnicola

bors[bot] commented 9 months ago

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here. For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

georust / gdal

Maintenance cleanup on `dataset.rs` #447

[X] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.

georust / gdal

Maintenance cleanup on `dataset.rs` #447

[X] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

[X] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.