removed clear_heading while sanitizing in loki_worker_t (#4288)

* removed clear_heading while sanitizing in loki_worker_t #1802 * updated CHANGELOG.MD #1802 --------- Co-authored-by: jehamannco <j.hamann@eclever.io> Co-authored-by: Nils <nilsnolde@proton.me>
valhalla · Sep 13, 2023 · 97cad06 · 97cad06
1 parent 7696735
commit 97cad06
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@
    * FIXED: fix crash in timedistancebssmatrix.cc  [#4244](https://github.com/valhalla/valhalla/pull/4244)
    * FIXED: missing protobuf CMake configuration to link abseil for protobuf >= 3.22.0 [#4207](https://github.com/valhalla/valhalla/pull/4207)
    * FIXED: broken links on the optimized route API page [#4260](https://github.com/valhalla/valhalla/pull/4260)
+   * FIXED: remove clearing of headings while calculating a matrix [#4288](https://github.com/valhalla/valhalla/pull/4288)
 * **Enhancement**
    * UPDATED: French translations, thanks to @xlqian [#4159](https://github.com/valhalla/valhalla/pull/4159)
    * CHANGED: -j flag for multithreaded executables to override mjolnir.concurrency [#4168](https://github.com/valhalla/valhalla/pull/4168)

diff --git a/src/loki/matrix_action.cc b/src/loki/matrix_action.cc
@@ -78,15 +78,9 @@ void loki_worker_t::init_matrix(Api& request) {
   if (options.sources_size() < 1) {
     throw valhalla_exception_t{121};
   };
-  for (auto& s : *options.mutable_sources()) {
-    s.clear_heading();
-  }
   if (options.targets_size() < 1) {
     throw valhalla_exception_t{122};
   };
-  for (auto& t : *options.mutable_targets()) {
-    t.clear_heading();
-  }
 
   // no locations!
   options.clear_locations();