[Tilesathome] close-areas.pl

Moshe Sayag msayag at gmail.com
Sun Apr 27 09:47:29 BST 2008


Hi,

I found the following code at the file close-areas.pl:

545 	if (scalar(@coastline_segments) == 1)
546 	{
547 	    make_way($coastline_segments[0]);
548 	}
549 	else
550 	{
551 	    make_multipolygon(\@coastline_segments);
552 	}


I notice that make_multipolygon is called even if coastline_segments
is empty (when nothing should actually be printed).
I suggest to change modify the code to

545 	if (scalar(@coastline_segments) == 1)
546 	{
547 	    make_way($coastline_segments[0]);
548 	}
549 	else if (scalar(@coastline_segments) > 1)
550 	{
551 	    make_multipolygon(\@coastline_segments);
552 	}

Moshe




More information about the Tilesathome mailing list