<div dir="ltr">Hello Even,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 3, 2015 at 5:13 PM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Le jeudi 03 septembre 2015 16:58:34, Rashad M a écrit :<br>
> Hello all,<br>
><br>
> I have an problem when reading metadata information of HDF5 dataset. some<br>
> of the values are not parsed properly by GDAL. Below is the relevant part<br>
> of output from gdalinfo<br>
><br>
><br>
> ....<br>
>   Azimuth_Processing_Number_of_Looks=<br>
>   Bits_per_Sample=<br>
> ....<br>
><br>
> S01_B001_Azimuth_Ramp_Code=<br>
><br>
> <garbage values and newline chars><br>
><br>
> ....<br>
> ....<br>
>  Sample_Format=INT<br>
>   Samples_per_Pixel=<br>
>   Satellite_Height=628440.5<br>
>   Satellite_ID=CSKS3<br>
> ....<br>
><br>
><br>
> I had looked into frmts/hdf5/hdf5dataset.cpp and all of these metadata are<br>
> of type H5T_NATIVE_UCHAR.<br>
><br>
> <a href="https://trac.osgeo.org/gdal/browser/branches/1.11/gdal/frmts/hdf5/hdf5datas" rel="noreferrer" target="_blank">https://trac.osgeo.org/gdal/browser/branches/1.11/gdal/frmts/hdf5/hdf5datas</a><br>
> et.cpp#L728<br>
><br>
> changing format specifier from char( %c) to int(%d) solves the issue for<br>
> me. But is there a better fix for this?<br>
<br>
</span>That looks like a reasonable fix. And the cast should be a (unsigned char*).<br>
<br>
%c should be used only for characters part of a string, but the string case is<br>
dealt before, so that must mean that those should be dealt as integer values.<br>
The _CHAR and _SCHAR above are also likely wrong.<br>
<br>
Could you raise a ticket about that in Trac ?<br></blockquote><div><br></div><div>Sure. The patch is for 1.11 branch. Should I generate the same for trunk also?<br><br></div><div>As we are on the subject of metadata in hdf5 dataset,  there is  Header too long. .. Truncated warning appearing when type is double which now has format specifier as %.16g<br><br></div><div>using simply %g or %lg fixed the issue on my side. If okay, I will include that too in the patch.<br><br></div><div>Thanks.<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
><br>
><br>
> OS: Fedora 20 x86_64<br>
> gdal branch: 1.11<br>
><br>
> svn info<br>
><br>
> Path: .<br>
> Working Copy Root Path: /home/mkanavat/sources/gdal_branch_1.11<br>
> URL: <a href="http://svn.osgeo.org/gdal/branches/1.11/gdal" rel="noreferrer" target="_blank">http://svn.osgeo.org/gdal/branches/1.11/gdal</a><br>
> Relative URL: ^/branches/1.11/gdal<br>
> Repository Root: <a href="http://svn.osgeo.org/gdal" rel="noreferrer" target="_blank">http://svn.osgeo.org/gdal</a><br>
> Repository UUID: f0d54148-0727-0410-94bb-9a71ac55c965<br>
> Revision: 30007<br>
> Node Kind: directory<br>
> Schedule: normal<br>
> Last Changed Author: rouault<br>
> Last Changed Rev: 30005<br>
> Last Changed Date: 2015-09-01 10:17:50 +0200 (Tue, 01 Sep 2015)<br>
<br>
--<br>
</span>Spatialys - Geospatial professional services<br>
<a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div><font face="arial, helvetica, sans-serif">Regards,<br>   Rashad</font></div></div>
</div></div></div>