[gdal-dev] Possible error in the commit 14983

Chaitanya kumar CH chaitanya.ch at gmail.com
Sat Jul 25 14:26:42 EDT 2009


Daniel,

You added some code in MIFFile::AddFieldNative() in mitab_miffile.cpp in the
revision 14983<http://trac.osgeo.org/gdal/changeset/14983/trunk/gdal/ogr/ogrsf_frmts/mitab/mitab_miffile.cpp>.
(
http://trac.osgeo.org/gdal/changeset/14983/trunk/gdal/ogr/ogrsf_frmts/mitab/mitab_miffile.cpp)

In the line 1742 of the new version, the statement "m_nVersion =
MAX(m_nVersion, 900);" comes after the "break;" making it unreachable. Can
you check if it is desirable? Shall I move it to above the "break;"
statement?

Best regards,
-- 
Chaitanya kumar CH.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.osgeo.org/pipermail/gdal-dev/attachments/20090725/d472d21e/attachment.html


More information about the gdal-dev mailing list