Adapt to feedback in #519

include/engine/api/base_parameters.hpp

@@ -7,6 +7,7 @@
 #include <boost/optional.hpp>
 
 #include <vector>
+#include <algorithm>
 
 namespace osrm
 {
@@ -32,7 +33,19 @@ struct BaseParameters
     {
         return (hints.empty() || hints.size() == coordinates.size()) &&
                (bearings.empty() || bearings.size() == coordinates.size()) &&
-               (radiuses.empty() || radiuses.size() == coordinates.size());
+               (radiuses.empty() || radiuses.size() == coordinates.size()) &&
+               std::all_of(bearings.begin(), bearings.end(),
+                           [](const boost::optional<Bearing> bearing_and_range)
+                           {
+                               if (bearing_and_range)
+                               {
+                                   return bearing_and_range->bearing >= 0 &&
+                                          bearing_and_range->bearing <= 360 &&
+                                          bearing_and_range->range >= 0 &&
+                                          bearing_and_range->range <= 180;
+                               }
+                               return true;
+                           });
     }
 };
 

include/engine/api/json_factory.hpp

@@ -76,7 +76,7 @@ util::json::Object makeRoute(const guidance::Route &route,
                              boost::optional<util::json::Value> geometry);
 
 util::json::Object
-makeWaypoint(const FixedPointCoordinate location, std::string &&way_name, const Hint &hint);
+makeWaypoint(const FixedPointCoordinate location, std::string &&name, const Hint &hint);
 
 util::json::Object makeRouteLeg(guidance::RouteLeg &&leg, util::json::Array &&steps);
 

include/engine/guidance/assemble_steps.hpp

@@ -35,12 +35,12 @@ inline StepManeuver stepManeuverFromGeometry(const extractor::TurnInstruction in
     const auto turn_coordinate = leg_geometry.locations[turn_index];
     const auto post_turn_coordinate = leg_geometry.locations[turn_index + 1];
 
-    const double pre_turn_heading =
+    const double pre_turn_bearing =
         util::coordinate_calculation::bearing(pre_turn_coordinate, turn_coordinate);
-    const double post_turn_heading =
+    const double post_turn_bearing =
         util::coordinate_calculation::bearing(turn_coordinate, post_turn_coordinate);
 
-    return StepManeuver{turn_coordinate, pre_turn_heading, post_turn_heading, instruction};
+    return StepManeuver{turn_coordinate, pre_turn_bearing, post_turn_bearing, instruction};
 }
 }
 

include/engine/guidance/route_step.hpp

@@ -22,7 +22,7 @@ namespace guidance
 struct RouteStep
 {
     unsigned name_id;
-    std::string way_name;
+    std::string name;
     double duration;
     double distance;
     extractor::TravelMode mode;

include/engine/guidance/step_maneuver.hpp

@@ -14,8 +14,8 @@ namespace guidance
 struct StepManeuver
 {
     util::FixedPointCoordinate location;
-    double heading_before;
-    double heading_after;
+    double bearing_before;
+    double bearing_after;
     extractor::TurnInstruction instruction;
 };