Use const auto for dataset pointer

* It's read-only operations

Signed-off-by: Ryan Friedman <[email protected]>

Author: Ryanf55

src/grid_map_geo.cpp

@@ -85,8 +85,7 @@ bool GridMapGeo::Load(const std::string &map_path, bool algin_terrain, const std
 
 bool GridMapGeo::initializeFromGdalDataset(const std::string &path, bool align_terrain) {
   GDALAllRegister();
-  GDALDatasetUniquePtr dataset;
-  dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen( path.c_str(), GA_ReadOnly )));
+  const auto dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen( path.c_str(), GA_ReadOnly )));
   if (!dataset) {
     std::cout << __func__ << "Failed to open '" << path << "'" << std::endl;
     return false;
@@ -221,7 +220,7 @@ bool GridMapGeo::initializeFromGdalDataset(const std::string &path, bool align_t
 
 bool GridMapGeo::addColorFromGeotiff(const std::string &path) {
   GDALAllRegister();
-  GDALDataset *dataset = (GDALDataset *)GDALOpen(path.c_str(), GA_ReadOnly);
+  const auto dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen( path.c_str(), GA_ReadOnly )));
   if (!dataset) {
     std::cout << __func__ << "Failed to open '" << path << "'" << std::endl;
     return false;
@@ -280,7 +279,7 @@ bool GridMapGeo::addColorFromGeotiff(const std::string &path) {
 
 bool GridMapGeo::addLayerFromGeotiff(const std::string &layer_name, const std::string &path) {
   GDALAllRegister();
-  GDALDataset *dataset = (GDALDataset *)GDALOpen(path.c_str(), GA_ReadOnly);
+  const auto dataset = GDALDatasetUniquePtr(GDALDataset::FromHandle(GDALOpen( path.c_str(), GA_ReadOnly )));
   if (!dataset) {
     std::cout << __func__ << "Failed to open '" << path << "'" << std::endl;
     return false;