PolygonHandler Read method bug

[GoogleCodeExporter]

After read all points content and GetZMValues, did not check all the contents 
are completely finished reading. 

// Trond Benum: We have now read all the parts, let's read optional Z and M 
values
// and populate Z in the coordinate before we start manipulating the segments
// We have to track corresponding optional M values and set them up in the 
// Geometries via ICoordinateSequence further down.
GetZMValues(file, totalRecordLength, ref totalRead, buffer, skippedList);

// sure to read all the contents
if (totalRead < totalRecordLength)
    file.ReadBytes((totalRecordLength - totalRead) * 2);

Original issue reported on code.google.com by roc....@gmail.com on 16 Nov 2013 at 1:44

Attachments:

• [Test Data.SHP](https://storage.googleapis.com/google-code-attachments/nettopologysuite/issue-167/comment-0/Test Data.SHP)

[GoogleCodeExporter]

your test data doesn't contains dbf file..

Original comment by diegogu...@gmail.com on 18 Nov 2013 at 10:42

• Changed state: Accepted

[GoogleCodeExporter]

There are 201 shapes in the Test Data.SHP file, you can read it by 
ShapefileReader..

Original comment by roc....@gmail.com on 18 Nov 2013 at 12:11

[GoogleCodeExporter]

my mistake, I'm considering DBF mandatory for the tests but actually isn't 
needed to read Z/M values... sorry.

Original comment by diegogu...@gmail.com on 18 Nov 2013 at 12:59

[GoogleCodeExporter]

added with commit 1087 
https://code.google.com/p/nettopologysuite/source/detail?r=1087

Original comment by diegogu...@gmail.com on 18 Nov 2013 at 1:46

• Changed state: Fixed