Bug report #15028
Use GDAL C API only
Status: | Closed | ||
---|---|---|---|
Priority: | Normal | ||
Assignee: | - | ||
Category: | Analysis library | ||
Affected QGIS version: | master | Regression?: | No |
Operating System: | Easy fix?: | No | |
Pull Request or Patch supplied: | No | Resolution: | |
Crashes QGIS or corrupts data: | No | Copied to github as #: | 22975 |
Description
The heatmap plugin seems to be the only use in-tree of the GDAL C++ API. Skimming quickly through it, it seems it could be converted to using the C API only.
Associated revisions
heatmap plugin: port to C-API (fixes #15028)
History
#1 Updated by Matthias Kuhn over 8 years ago
What's the advantage of this?
#2 Updated by Even Rouault over 8 years ago
(Original discussion: https://lists.osgeo.org/pipermail/gdal-dev/2016-June/044586.html + https://lists.osgeo.org/pipermail/gdal-dev/2016-June/044589.html)
The GDAL C API/ABI is more stable than the C++ one. And from a design point of view, it would be best if the use of GDAL API by QGIS would be consistent through the code base
#3 Updated by Jürgen Fischer over 8 years ago
- Status changed from Open to Closed
Fixed in changeset 186036659e31f416751f40ffc872da1d23996704.
#4 Updated by Jürgen Fischer over 8 years ago
- Status changed from Closed to In Progress
Even Rouault wrote:
(Original discussion: https://lists.osgeo.org/pipermail/gdal-dev/2016-June/044586.html + https://lists.osgeo.org/pipermail/gdal-dev/2016-June/044589.html)
Nope, original discussion is https://lists.osgeo.org/pipermail/qgis-developer/2008-February/003295.html leading to fb8424d ;)
#5 Updated by Jürgen Fischer over 8 years ago
- Status changed from In Progress to Closed
#6 Updated by Giovanni Manghi almost 8 years ago
- Category changed from 114 to Analysis library