From 84b35f47a10c2798b279faebd8f8f1a4b0e0121c Mon Sep 17 00:00:00 2001
From: DennisOSRM <luxen@kit.edu>
Date: Tue, 14 Feb 2012 17:17:11 +0100
Subject: [PATCH] Fixes test cucumber --tags @basic:63 and issue #105

---
 Descriptors/DescriptionFactory.cpp | 35 +++++++++++++++++-------------
 1 file changed, 20 insertions(+), 15 deletions(-)

diff --git a/Descriptors/DescriptionFactory.cpp b/Descriptors/DescriptionFactory.cpp
index 761d474a6..c723b9e18 100644
--- a/Descriptors/DescriptionFactory.cpp
+++ b/Descriptors/DescriptionFactory.cpp
@@ -95,27 +95,32 @@ void DescriptionFactory::Run(const unsigned zoomLevel, const unsigned duration)
             indexOfSegmentBegin = i;
         }
     }
+//    INFO("#segs: " << pathDescription.size());
 
     //Post-processing to remove empty or nearly empty path segments
-    if(0. == startPhantom.ratio || 0 == pathDescription[0].length) {
-        if(pathDescription.size() > 2)
-            pathDescription.erase(pathDescription.begin());
-        pathDescription[0].turnInstruction = TurnInstructions.HeadOn;
-        pathDescription[0].necessary = true;
-        startPhantom.nodeBasedEdgeNameID = pathDescription[0].nameID;
-    } else {
-        pathDescription[0].duration *= startPhantom.ratio;
-    }
-    if(1. == targetPhantom.ratio || 0 == pathDescription.back().length) {
-        if(pathDescription.size() > 2)
+    if(0 == pathDescription.back().length) {
+//        INFO("#segs: " << pathDescription.size() << ", last ratio: " << targetPhantom.ratio << ", length: " << pathDescription.back().length);
+        if(pathDescription.size() > 2){
             pathDescription.pop_back();
-        pathDescription.back().necessary = true;
-        pathDescription.back().turnInstruction = TurnInstructions.NoTurn;
-        targetPhantom.nodeBasedEdgeNameID = (pathDescription.end()-2)->nameID;
-//        INFO("Deleting last turn instruction");
+            pathDescription.back().necessary = true;
+            pathDescription.back().turnInstruction = TurnInstructions.NoTurn;
+            targetPhantom.nodeBasedEdgeNameID = (pathDescription.end()-2)->nameID;
+//            INFO("Deleting last turn instruction");
+        }
     } else {
         pathDescription[indexOfSegmentBegin].duration *= (1.-targetPhantom.ratio);
     }
+    if(0 == pathDescription[0].length) {
+        if(pathDescription.size() > 2) {
+            pathDescription.erase(pathDescription.begin());
+            pathDescription[0].turnInstruction = TurnInstructions.HeadOn;
+            pathDescription[0].necessary = true;
+            startPhantom.nodeBasedEdgeNameID = pathDescription[0].nameID;
+//            INFO("Deleting first turn instruction, ratio: " << startPhantom.ratio << ", length: " << pathDescription[0].length);
+        }
+    } else {
+        pathDescription[0].duration *= startPhantom.ratio;
+    }
 
     //Generalize poly line
     dp.Run(pathDescription, zoomLevel);