<br><br><div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">So, the code for Reorder Segments seems to rely on all the segments
<br>facing the same way, or it won't work properly - it'll order the parts<br>that are facing the same way, but not the whole thing (as would be<br>expected, you can't order them when they aren't facing the same way).
<br>So, to fix this (from my point of view anyway), I've gone and written a<br>new function I've called "Direct all way segments", that seems to solve<br>all the "Unordered ways" warnings (that are really unordered ways, and
<br>not other issues like roads being split in two).<br><br>What it essentially does is to Sort all the segments in a way<br>irresepective of the direction they're facing, it then makes all of them<br>face the same direction as the one on the end, it then reorders the
<br>segments in the way. I think this is considerably easier than the<br>previous method which involved selecting each segment, switching it then<br>reordering them.</blockquote><div><br>Does it fix the lollipop problem? This is when the final segment of a way ends on one of the intermediate nodes in the way. These should be orderable (if that is a word), because you can start either from the node that has only segment from it, or from the node that has 3 segments as long as you do the loop first.
<br><br><br> </div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I'm not sure what process there is for reviewing patches (if any), or
<br>what I should do with this code.<br><br></blockquote></div><br>Posting it on this list should be sufficient. It used to be that Imi needed to apply it to JOSM, but there is now another JOSM maintainer on this list (deelkar?).
<br><br>-- <br>Robert Hart