Fix SetLinearBinning call in HFA driver to set binning type properly
|Reported by:||Sam Gillingham||Owned by:||warmerdam|
We have noticed that some parts of the HFA driver are inconsistent when setting the binning type. When setting the STATISTICS_HISTOBINVALUES metadata item the type of the layer (thematic or athematic) is taken into account to set it correctly - see hfaopen.cpp line 2877. This patch fixes another place where the binning type was hardcoded so the SetLinearBinning call can be used safely. Imagine gets confused if the binning type isn't set to direct for thematic and linear for athematic (see https://trac.osgeo.org/gdal/ticket/5066).
There are other places in the HFA driver that probably need this fix, and also change binFunction -> binFunctionType since this is a mistake (see the default data dictionary). Even, are you happy for another patch to be submitted to address these areas?