assume roads have sidewalks if not explicitely tagged, see discussion graphhopper#476

Author: Peter

core/src/main/java/com/graphhopper/routing/util/FootFlagEncoder.java

@@ -113,10 +113,9 @@ public FootFlagEncoder( int speedBits, double speedFactor )
         avoidHighwayTags.add("secondary_link");
         avoidHighwayTags.add("tertiary");
         avoidHighwayTags.add("tertiary_link");
-        
+
         // for now no explicit avoiding #257
         //avoidHighwayTags.add("cycleway"); 
-
         allowedHighwayTags.addAll(safeHighwayTags);
         allowedHighwayTags.addAll(avoidHighwayTags);
         allowedHighwayTags.add("cycleway");
@@ -367,9 +366,7 @@ void collect( OSMWay way, TreeMap<Double, Integer> weightToPrioMap )
             }
         } else if (maxSpeed > 50 || avoidHighwayTags.contains(highway))
         {
-            if (way.hasTag("sidewalk", sidewalks))
-                weightToPrioMap.put(45d, AVOID_IF_POSSIBLE.getValue());
-            else if (way.hasTag("sidewalk", "no"))
+            if (way.hasTag("sidewalk", "no"))
                 weightToPrioMap.put(45d, WORST.getValue());
             else
                 weightToPrioMap.put(45d, REACH_DEST.getValue());

core/src/test/java/com/graphhopper/routing/util/FootFlagEncoderTest.java

@@ -221,7 +221,7 @@ public void testPriority()
         way.clearTags();
         way.setTag("highway", "primary");
         way.setTag("sidewalk", "yes");
-        assertEquals(PriorityCode.AVOID_IF_POSSIBLE.getValue(), footEncoder.handlePriority(way, 0));
+        assertEquals(PriorityCode.REACH_DEST.getValue(), footEncoder.handlePriority(way, 0));
 
         way.clearTags();
         way.setTag("highway", "cycleway");