Hi Gerd orderByDecreasingArea only has a param declaration and single use in MapSplitter and its value transforms to MapArea.splitPolygonsIntoArea so I think it is better as it is.
I had some thoughts on a simple/efficient algo for MultiPolygonRelation hole cutting - It won't produce the optimized cuts that yours is aiming for. If you are interested, I'll formulate some pseudo-code. Merge the branch whenever is best for you. Ticker On Tue, 2017-02-07 at 13:12 +0000, Gerd Petermann wrote: > Hi Ticker, > > thanks, I am still working a new algos to replace parts of the > existing code in MultipolygonRelation, > esp. that for createContainsMatrix(). I hope to finish the first > patch tomorrow. > If that takes much longer I plan to merge the branch into trunk > first. > Reg. refactoring: What do you think about adding > orderByDecreasingArea as a (final) field in MapSplitter? > > Gerd > > > ________________________________________ > Von: mkgmap-dev <mkgmap-dev-boun...@lists.mkgmap.org.uk> im Auftrag > von Ticker Berkin <rwb-mkg...@jagit.co.uk> > Gesendet: Dienstag, 7. Februar 2017 13:45:25 > An: mkgmap-dev@lists.mkgmap.org.uk > Betreff: Re: [mkgmap-dev] r3784 produces large img files than r3773 > > Done > > On Tue, 2017-02-07 at 12:17 +0000, Gerd Petermann wrote: > > Hi Ticker, > > > > that's what I expected. Most shapes do not have too many points > > after line simplification. > > If I got that right most of the code in PolygonSplitterFilter is > > now > > obsolete. I think it would be > > better to move the functionality of PolygonSplitIfNeededFilter into > > it so that the history is kept. > > > > Gerd > > _______________________________________________ > mkgmap-dev mailing list > mkgmap-dev@lists.mkgmap.org.uk > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > _______________________________________________ > mkgmap-dev mailing list > mkgmap-dev@lists.mkgmap.org.uk > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
