Switching to use mode of fix instead of status to be more robust to c…

…hanges in API
swri-robotics · May 9, 2024 · e6ee597 · e6ee597
1 parent 4e61959
commit e6ee597
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/gpsd_client/src/client.cpp b/gpsd_client/src/client.cpp
@@ -177,12 +177,12 @@ namespace gpsd_client
       }
 #if GPSD_API_MAJOR_VERSION >= 10
 #ifdef STATUS_FIX
-      if ((p->fix.status != STATUS_NO_FIX) && !(check_fix_by_variance_ && std::isnan(p->fix.epx)))
+      if (((p->fix.mode == MODE_2D) || (p->fix.mode == MODE_3D)) && !(check_fix_by_variance_ && std::isnan(p->fix.epx)))
 #else
-      if ((p->fix.status != STATUS_NO_FIX) && !(check_fix_by_variance_ && std::isnan(p->fix.epx)))
+      if (((p->fix.mode == MODE_2D) || (p->fix.mode == MODE_3D)) && !(check_fix_by_variance_ && std::isnan(p->fix.epx)))
 #endif
 #else
-      if ((p->status != STATUS_NO_FIX) && !(check_fix_by_variance_ && std::isnan(p->fix.epx)))
+      if (((p->fix.mode == MODE_2D) || (p->fix.mode == MODE_3D)) && !(check_fix_by_variance_ && std::isnan(p->fix.epx)))
 #endif
       {
         status.status = 0; // FIXME: gpsmm puts its constants in the global