[mkgmap-dev] area_size() function to be used in points file

Thu Apr 19 11:46:43 BST 2018

I was being a bit too severe. Zero and negative width/height
subdivisions previously were allowed through without being noticed,
with the negative quietly overflowing looking positive.

The attached patch downgrades the message for zero size to a warning
but flags an error for neg.

Regards
Ticker

On Thu, 2018-04-19 at 09:49 +0000, Gerd Petermann wrote:
> Hi Joris,
> 
> thanks for testing. The severe messages were introduced with the
> latest patches from Ticker:
> http://www.mkgmap.org.uk/websvn/revision.php?repname=mkgmap&rev=4167
> 
> The added source in Subdivision.java:
>                 if (w <= 0) { // think can get this from
> empty/unbounded maps
>                         log.error("Subdivision neg width ", w,
> "min=", area.getMinLong(), "mid=", longitude, "max=",
> area.getMaxLong(), "shift=", shift, "mask=", mask, "@", getCenter());
>                         w = 0x7fff;
>                 }
> 
> @Ticker:
> I don't know why we have those new messages?
> 
> Gerd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: negSubDiv.patch
Type: text/x-patch
Size: 1569 bytes
Desc: not available
URL: <http://www.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20180419/f54987cf/attachment.bin>