Android-builders-club/OsmAnd
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix critical bug with roundabout

Browse source
This commit is contained in:
Victor Shcherb 2012-10-18 18:42:44 +02:00
parent 027d49ae24
commit ce564e82cf
1 changed files with 17 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
DataExtractionOSM/src/net/osmand/router/RoutingContext.java
View file

@ -300,7 +300,7 @@ public class RoutingContext {
for (RouteDataObject ro : routes) { for (RouteDataObject ro : routes) {
for (int i = 0; i < ro.pointsX.length; i++) { for (int i = 0; i < ro.pointsX.length; i++) {
if (ro.getPoint31XTile(i) == x31 && ro.getPoint31YTile(i) == y31) { if (ro.getPoint31XTile(i) == x31 && ro.getPoint31YTile(i) == y31) {
excludeDuplications.put(ro.id, ro); excludeDuplications.put(calcRouteId(ro, i), ro);
RouteSegment segment = new RouteSegment(ro, i); RouteSegment segment = new RouteSegment(ro, i);
segment.next = original; segment.next = original;
original = segment; original = segment;
@ -561,6 +561,10 @@ public class RoutingContext {
} }
private static long calcRouteId(RouteDataObject o, int ind) {
return (o.getId() << 10) + ind;
}
public static class RoutingSubregionTile { public static class RoutingSubregionTile {
public final RouteSubregion subregion; public final RouteSubregion subregion;
@ -614,9 +618,9 @@ public class RoutingContext {
RouteSegment segment = routes.get(l); RouteSegment segment = routes.get(l);
while (segment != null) { while (segment != null) {
RouteDataObject ro = segment.road; RouteDataObject ro = segment.road;
RouteDataObject toCmp = excludeDuplications.get(ro.id); RouteDataObject toCmp = excludeDuplications.get(calcRouteId(ro, segment.getSegmentStart()));
if (toCmp == null || toCmp.getPointsLength() < ro.getPointsLength()) { if (toCmp == null || toCmp.getPointsLength() < ro.getPointsLength()) {
excludeDuplications.put(ro.id, ro); excludeDuplications.put(calcRouteId(ro, segment.getSegmentStart()), ro);
RouteSegment s = new RouteSegment(ro, segment.getSegmentStart()); RouteSegment s = new RouteSegment(ro, segment.getSegmentStart());
s.next = original; s.next = original;
original = s; original = s;
@ -631,18 +635,21 @@ public class RoutingContext {
ctx.timeToLoad += (System.nanoTime() - nanoTime); ctx.timeToLoad += (System.nanoTime() - nanoTime);
if (res != null) { if (res != null) {
for (RouteDataObject ro : res) { for (RouteDataObject ro : res) {
RouteDataObject toCmp = excludeDuplications.get(ro.id);
boolean accept = ro != null && (toCmp == null || toCmp.getPointsLength() < ro.getPointsLength()); boolean accept = ro != null;
if (ctx != null && accept) { if (ctx != null) {
accept = ctx.getRouter().acceptLine(ro); accept = ctx.getRouter().acceptLine(ro);
} }
if (accept) { if (accept) {
excludeDuplications.put(ro.id, ro);
for (int i = 0; i < ro.pointsX.length; i++) { for (int i = 0; i < ro.pointsX.length; i++) {
if (ro.getPoint31XTile(i) == x31 && ro.getPoint31YTile(i) == y31) { if (ro.getPoint31XTile(i) == x31 && ro.getPoint31YTile(i) == y31) {
RouteDataObject toCmp = excludeDuplications.get(calcRouteId(ro, i));
if (toCmp == null || toCmp.getPointsLength() < ro.getPointsLength()) {
RouteSegment segment = new RouteSegment(ro, i); RouteSegment segment = new RouteSegment(ro, i);
segment.next = original; segment.next = original;
original = segment; original = segment;
excludeDuplications.put(calcRouteId(ro, i), ro);
}
} }
} }
} }