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.