logo separator

[mkgmap-dev] [PATCH v1] make DP filter avoid removing points located at nodes

From Felix Hartmann extremecarver at googlemail.com on Fri Nov 27 00:06:36 GMT 2009

Mark Burton wrote:
> Hi Johann,
>> I have now looked into the code and must say, I don't understand why it 
>> works. Previous for each node was created a CoordNode. Now instead a 
>> flag is set. I see the difference: The flag is copied, the class 
>> obviously not.
>> But I didn't find the place, where this copy occurs. Could you point my 
>> nose to the file and line number, where the line is duplicated? The only 
>> place I found is in the round coord filter class (line 55), where it is 
>> handled correctly.
>> So why your patch works as expected and the original solution not??
> As I said in a previous posting on this topic:
> mb> Whenever a routable way gets split to limit the number of points or
> mb> nodes it makes a new list of points so any changes to the points in the
> mb> new list will not be visible in the original list.
> So, wherever you see splitWay() that's where the "copying" occurs.
>> I have another idea of solving this problem.
>> Instead of do a new Coord or CoordNode call coord.copy(). The function 
>> could be overwritten in the CoordNode class to return a CoordNode 
>> object. (BTW. This should be done in the CoordPOI class too). So a 
>> copied line should contain afterwards the same types as the original.
> Doesn't the suggested patch do the job? (Felix, you reported it wasn't
> working, is that still the case?)
Well since introducing the round-cords it is not working for me anymore. 
nothing changed here.
> Actually, I'm not keen on the CoordNode thing at all but it
> works well enough to not warrant spending effort on it while there is
> so much other stuff to be fixed.
> Cheers,
> Mark
> _______________________________________________
> mkgmap-dev mailing list
> mkgmap-dev at lists.mkgmap.org.uk
> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20091127/ab5231c5/attachment.html 

More information about the mkgmap-dev mailing list