Backport MITAB 1.6.1 bug fixes to GDAL 1.4 branch

[Daniel Morissette]

The OGR-MITAB driver in the GDAL 1.4.x stable branch is based on MITAB 1.5.1. There have been a couple of important bug fixes in MITAB 1.6.0 and 1.6.1 that should be backported, without necessarily upgrading to the full MITAB 1.6.1 release because it contains a bunch of non-trivial changes in the spatial index handling code that it is not desirable to commit in a stable branch.

Here is the list of MITAB 1.6.0/1.6.1 bug fixes to backport:

• Fixed problem writing collections where MITAB was sometimes trying to read past EOF in write mode (​http://bugzilla.maptools.org/show_bug.cgi?id=1657).

• Fixed another problem writing collections when the header of objects part of a collection were split on multiple blocks (​http://bugzilla.maptools.org/show_bug.cgi?id=1663)

• Added missing NULL pointer checks in SetPenFromStyleString(), SetBrushFromStyleString() and SetSymbolFromStyleString() (​http://bugzilla.maptools.org/show_bug.cgi?id=1670)

• Fixed problem with null brush bg color becoming black when parsing style strings (​http://bugzilla.maptools.org/show_bug.cgi?id=1603)

[Daniel Morissette]

Working on this now.

[Daniel Morissette]

MITAB bug 1603 backported in SVN r11098

MITAB bug 1670 backported in SVN r11101

The other two will be more tricky but I have to leave now. I will backport them before the 1.4.1 beta on Tuesday.

[Daniel Morissette]

MITAB bug 1663 backported in SVN r11167

MITAB bug 1657 backported in SVN r11168

Closing this ticket as FIXED.