[mkgmap-dev] [PATCH] Again NullPointerException

Sun Feb 12 16:20:16 GMT 2012

Gerd,

one more question about the patch:
Are the min/max lat/lon variables used? I think they can be removed but 
I fear that I have overseen something important again.... ;-)?

WanMil

>>   >  >  Hi,
>>   >  >
>>   >  >  as a result of previous discussions here is th new patch.
>>   >  >
>>   >  >  http://gis.19327.n5.nabble.com/file/n5471749/verify_boundary_v3.patch
>>   >  >  verify_boundary_v3.patch
>>   >  >
>>   >  >  Changes:
>>   >  >  Verify if rounding errors change the direction of a way
>>   >  >  (clockwise/counterclockwise order), if that happens,
>>   >  >  change the order so that Way.clockwise() returns the wanted result.
>>   >  >
>>   >  >  The patch introduces a new method
>>   >  >
>>   >  >  Java2DConverter.verifiedAreaToShapes()
>>   >  >
>>   >  >  instead of changing the existing
>>   >  >
>>   >  >  Java2DConverter.areaToShapes(area)
>>   >  >
>>   >  >  I did this because Java2DConverter.areaToShapes(area) is called in
>> other
>>   >  >  places and I wanted to avoid side effects. Maybe someone who knows the
>>   >  >  sources in
>>   >  >  PolygonSplitterBase.java and PolygonClipper.java
>>   >  >  can look at this and see if they benefit also from the verified
>> routine?
>>   >  >
>>   >  >  Gerd