<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>Hi WanMil,<br><br>I just found Path2D.append(Shape s, boolean connect) which seems to do the same<br>as the addToPath() method. The advantage compared to multiple Area.add() calls is that<br>each Area.add() call also tries and remove obsolete points and dangling edges.<br>The more edges an Area has, the more time is wasted with this.<br><br>The trick can probably also be used in all MultipolygonRelation classes. Working on a patch...<br><br>Gerd<br><br><div>&gt; Date: Thu, 23 Jan 2014 13:00:37 +0100<br>&gt; From: wmgcnfg@web.de<br>&gt; To: mkgmap-dev@lists.mkgmap.org.uk<br>&gt; Subject: Re: [mkgmap-dev] Commit: r2971: Avoid precalculation of area size for sea polygons        created by the SeaGenerator.<br>&gt; <br>&gt; Hi Gerd,<br>&gt; <br>&gt; thanks for the hint. It's fixed now.<br>&gt; <br>&gt; I have no time to test your patch now but it sounds good :-)<br>&gt; <br>&gt; WanMil<br>&gt; <br>&gt; <br>&gt; &gt; Hi WanMil,<br>&gt; &gt;<br>&gt; &gt; this doesn't seem to work. I still see the tag in the precompiled<br>&gt; &gt; sea data:<br>&gt; &gt;<br>&gt; &gt;                 &lt;tag k="natural" v="sea"/&gt;<br>&gt; &gt;                 &lt;tag k="mkgmap:cache_area_size" v="1048709826.5"/&gt;<br>&gt; &gt;<br>&gt; &gt; I don't fully understand the idea. The current implementation seems to<br>&gt; &gt; calculate the value when precompiling and avoids recalculation when reading<br>&gt; &gt; the precompiled data.<br>&gt; &gt;<br>&gt; &gt; Gerd<br>&gt; &gt; P.S. I found some nice performance improvements for the precompSeaMerger.<br>&gt; &gt; Reduced run time from 2200 sec to  ~330sec.<br>&gt; &gt; The most important change is to use BoundaryUtil.addToPath() instead of<br>&gt; &gt; Area.add().<br>&gt; &gt; See attached patch for the high-prec-branch.<br>&gt; &gt; I'll probably move the code for addToPath() from BoundaryUtil to<br>&gt; &gt; Java2DConverter.<br>&gt; &gt; PrecompSeaMerger.java.patch<br>&gt; &gt; &lt;http://gis.19327.n5.nabble.com/file/n5794046/PrecompSeaMerger.java.patch&gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt; svn commit wrote<br>&gt; &gt;&gt; Version mkgmap-r2971 was committed by wanmil on Thu, 16 Jan 2014<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; Avoid precalculation of area size for sea polygons created by the<br>&gt; &gt;&gt; SeaGenerator.<br>&gt; &gt;&gt;<br>&gt; &gt;&gt; For all sea polygons the area size was added as tag which increases the<br>&gt; &gt;&gt; file size of the sea tiles but does not really help. The size only<br>&gt; &gt;&gt; calculated the size of the sea in one sea tile and it's a bit random where<br>&gt; &gt;&gt; the borders of a sea tile are. So the area size is not usefull.<br>&gt; &gt;&gt; _______________________________________________<br>&gt; &gt;&gt; mkgmap-dev mailing list<br>&gt; &gt;<br>&gt; &gt;&gt; mkgmap-dev@.org<br>&gt; &gt;<br>&gt; &gt;&gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt;<br>&gt; &gt; --<br>&gt; &gt; View this message in context: http://gis.19327.n5.nabble.com/Commit-r2971-Avoid-precalculation-of-area-size-for-sea-polygons-created-by-the-SeaGenerator-tp5793369p5794046.html<br>&gt; &gt; Sent from the Mkgmap Development mailing list archive at Nabble.com.<br>&gt; &gt; _______________________________________________<br>&gt; &gt; mkgmap-dev mailing list<br>&gt; &gt; mkgmap-dev@lists.mkgmap.org.uk<br>&gt; &gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br>&gt; &gt;<br>&gt; <br>&gt; _______________________________________________<br>&gt; mkgmap-dev mailing list<br>&gt; mkgmap-dev@lists.mkgmap.org.uk<br>&gt; http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br></div>                                               </div></body>
</html>