[ABSL] Purge common::optional. (#1339)

cartographer-project · Jul 27, 2018 · 4990d4d · 4990d4d
1 parent dafb414
commit 4990d4d
Show file tree

Hide file tree

Showing 23 changed files with 47 additions and 168 deletions.
diff --git a/cartographer/BUILD.bazel b/cartographer/BUILD.bazel
@@ -95,12 +95,12 @@ cc_library(
     deps = [
         ":cc_protos",
         "@boost//:iostreams",
+        "@com_google_absl//absl/types:optional",
         "@com_google_glog//:glog",
         "@org_cairographics_cairo//:cairo",
         "@org_ceres_solver_ceres_solver//:ceres",
         "@org_lua_lua//:lua",
         "@org_tuxfamily_eigen//:eigen",
-        "@com_google_absl//absl/strings",
     ],
 )
 

diff --git a/cartographer/cloud/internal/client/pose_graph_stub.cc b/cartographer/cloud/internal/client/pose_graph_stub.cc
@@ -93,7 +93,7 @@ PoseGraphStub::GetTrajectoryNodePoses() const {
   CHECK(client.Write(request));
   mapping::MapById<mapping::NodeId, mapping::TrajectoryNodePose> node_poses;
   for (const auto& node_pose : client.response().node_poses()) {
-    common::optional<mapping::TrajectoryNodePose::ConstantPoseData>
+    absl::optional<mapping::TrajectoryNodePose::ConstantPoseData>
         constant_pose_data;
     if (node_pose.has_constant_pose_data()) {
       constant_pose_data = mapping::TrajectoryNodePose::ConstantPoseData{

diff --git a/cartographer/cloud/internal/handlers/receive_global_slam_optimizations_handler.h b/cartographer/cloud/internal/handlers/receive_global_slam_optimizations_handler.h
@@ -39,7 +39,7 @@ class ReceiveGlobalSlamOptimizationsHandler
   void OnFinish() override;
 
  private:
-  common::optional<int> subscription_index_;
+  absl::optional<int> subscription_index_;
 };
 
 }  // namespace handlers

diff --git a/cartographer/common/optional.h b/cartographer/common/optional.h
diff --git a/cartographer/common/optional_test.cc b/cartographer/common/optional_test.cc
diff --git a/cartographer/mapping/2d/tsdf_2d.cc b/cartographer/mapping/2d/tsdf_2d.cc
@@ -16,6 +16,8 @@
 
 #include "cartographer/mapping/2d/tsdf_2d.h"
 
+#include "cartographer/common/make_unique.h"
+
 namespace cartographer {
 namespace mapping {
 

diff --git a/cartographer/mapping/internal/2d/local_trajectory_builder_2d.cc b/cartographer/mapping/internal/2d/local_trajectory_builder_2d.cc
@@ -187,7 +187,7 @@ LocalTrajectoryBuilder2D::AddRangeData(
 
   if (num_accumulated_ >= options_.num_accumulated_range_data()) {
     const common::Time current_sensor_time = synchronized_data.time;
-    common::optional<common::Duration> sensor_duration;
+    absl::optional<common::Duration> sensor_duration;
     if (last_sensor_time_.has_value()) {
       sensor_duration = current_sensor_time - last_sensor_time_.value();
     }
@@ -213,7 +213,7 @@ LocalTrajectoryBuilder2D::AddAccumulatedRangeData(
     const common::Time time,
     const sensor::RangeData& gravity_aligned_range_data,
     const transform::Rigid3d& gravity_alignment,
-    const common::optional<common::Duration>& sensor_duration) {
+    const absl::optional<common::Duration>& sensor_duration) {
   if (gravity_aligned_range_data.returns.empty()) {
     LOG(WARNING) << "Dropped empty horizontal range data.";
     return nullptr;

diff --git a/cartographer/mapping/internal/2d/local_trajectory_builder_2d.h b/cartographer/mapping/internal/2d/local_trajectory_builder_2d.h
@@ -80,7 +80,7 @@ class LocalTrajectoryBuilder2D {
   std::unique_ptr<MatchingResult> AddAccumulatedRangeData(
       common::Time time, const sensor::RangeData& gravity_aligned_range_data,
       const transform::Rigid3d& gravity_alignment,
-      const common::optional<common::Duration>& sensor_duration);
+      const absl::optional<common::Duration>& sensor_duration);
   sensor::RangeData TransformToGravityAlignedFrameAndFilter(
       const transform::Rigid3f& transform_to_gravity_aligned_frame,
       const sensor::RangeData& range_data) const;
@@ -112,9 +112,9 @@ class LocalTrajectoryBuilder2D {
   int num_accumulated_ = 0;
   sensor::RangeData accumulated_range_data_;
 
-  common::optional<std::chrono::steady_clock::time_point> last_wall_time_;
-  common::optional<double> last_thread_cpu_time_seconds_;
-  common::optional<common::Time> last_sensor_time_;
+  absl::optional<std::chrono::steady_clock::time_point> last_wall_time_;
+  absl::optional<double> last_thread_cpu_time_seconds_;
+  absl::optional<common::Time> last_sensor_time_;
 
   RangeDataCollator range_data_collator_;
 };

diff --git a/cartographer/mapping/internal/2d/pose_graph_2d.cc b/cartographer/mapping/internal/2d/pose_graph_2d.cc
@@ -845,7 +845,7 @@ MapById<NodeId, TrajectoryNodePose> PoseGraph2D::GetTrajectoryNodePoses()
   MapById<NodeId, TrajectoryNodePose> node_poses;
   common::MutexLocker locker(&mutex_);
   for (const auto& node_id_data : data_.trajectory_nodes) {
-    common::optional<TrajectoryNodePose::ConstantPoseData> constant_pose_data;
+    absl::optional<TrajectoryNodePose::ConstantPoseData> constant_pose_data;
     if (node_id_data.data.constant_data != nullptr) {
       constant_pose_data = TrajectoryNodePose::ConstantPoseData{
           node_id_data.data.constant_data->time,

diff --git a/cartographer/mapping/internal/3d/local_trajectory_builder_3d.cc b/cartographer/mapping/internal/3d/local_trajectory_builder_3d.cc
@@ -198,7 +198,7 @@ LocalTrajectoryBuilder3D::AddRangeData(
   ++num_accumulated_;
 
   if (num_accumulated_ >= options_.num_accumulated_range_data()) {
-    common::optional<common::Duration> sensor_duration;
+    absl::optional<common::Duration> sensor_duration;
     if (last_sensor_time_.has_value()) {
       sensor_duration = current_sensor_time - last_sensor_time_.value();
     }
@@ -237,7 +237,7 @@ std::unique_ptr<LocalTrajectoryBuilder3D::MatchingResult>
 LocalTrajectoryBuilder3D::AddAccumulatedRangeData(
     const common::Time time,
     const sensor::RangeData& filtered_range_data_in_tracking,
-    const common::optional<common::Duration>& sensor_duration) {
+    const absl::optional<common::Duration>& sensor_duration) {
   if (filtered_range_data_in_tracking.returns.empty()) {
     LOG(WARNING) << "Dropped empty range data.";
     return nullptr;

diff --git a/cartographer/mapping/internal/3d/local_trajectory_builder_3d.h b/cartographer/mapping/internal/3d/local_trajectory_builder_3d.h
@@ -78,7 +78,7 @@ class LocalTrajectoryBuilder3D {
   std::unique_ptr<MatchingResult> AddAccumulatedRangeData(
       common::Time time,
       const sensor::RangeData& filtered_range_data_in_tracking,
-      const common::optional<common::Duration>& sensor_duration);
+      const absl::optional<common::Duration>& sensor_duration);
 
   std::unique_ptr<InsertionResult> InsertIntoSubmap(
       common::Time time, const sensor::RangeData& filtered_range_data_in_local,
@@ -107,13 +107,13 @@ class LocalTrajectoryBuilder3D {
 
   int num_accumulated_ = 0;
   sensor::RangeData accumulated_range_data_;
-  common::optional<std::chrono::steady_clock::time_point> last_wall_time_;
+  absl::optional<std::chrono::steady_clock::time_point> last_wall_time_;
 
-  common::optional<double> last_thread_cpu_time_seconds_;
+  absl::optional<double> last_thread_cpu_time_seconds_;
 
   RangeDataCollator range_data_collator_;
 
-  common::optional<common::Time> last_sensor_time_;
+  absl::optional<common::Time> last_sensor_time_;
 };
 
 }  // namespace mapping

diff --git a/cartographer/mapping/internal/3d/pose_graph_3d.cc b/cartographer/mapping/internal/3d/pose_graph_3d.cc
@@ -888,7 +888,7 @@ MapById<NodeId, TrajectoryNodePose> PoseGraph3D::GetTrajectoryNodePoses()
   MapById<NodeId, TrajectoryNodePose> node_poses;
   common::MutexLocker locker(&mutex_);
   for (const auto& node_id_data : data_.trajectory_nodes) {
-    common::optional<TrajectoryNodePose::ConstantPoseData> constant_pose_data;
+    absl::optional<TrajectoryNodePose::ConstantPoseData> constant_pose_data;
     if (node_id_data.data.constant_data != nullptr) {
       constant_pose_data = TrajectoryNodePose::ConstantPoseData{
           node_id_data.data.constant_data->time,

diff --git a/cartographer/mapping/internal/optimization/optimization_problem_3d.h b/cartographer/mapping/internal/optimization/optimization_problem_3d.h
@@ -24,7 +24,7 @@
 
 #include "Eigen/Core"
 #include "Eigen/Geometry"
-#include "cartographer/common/optional.h"
+#include "absl/types/optional.h"
 #include "cartographer/common/port.h"
 #include "cartographer/common/time.h"
 #include "cartographer/mapping/id.h"

diff --git a/cartographer/mapping/pose_graph_interface.h b/cartographer/mapping/pose_graph_interface.h
@@ -20,7 +20,7 @@
 #include <chrono>
 #include <vector>
 
-#include "cartographer/common/optional.h"
+#include "absl/types/optional.h"
 #include "cartographer/mapping/id.h"
 #include "cartographer/mapping/submaps.h"
 #include "cartographer/transform/rigid_transform.h"
@@ -61,7 +61,7 @@ class PoseGraphInterface {
       double rotation_weight;
     };
     std::vector<LandmarkObservation> landmark_observations;
-    common::optional<transform::Rigid3d> global_landmark_pose;
+    absl::optional<transform::Rigid3d> global_landmark_pose;
   };
 
   struct SubmapPose {
@@ -77,7 +77,7 @@ class PoseGraphInterface {
   struct TrajectoryData {
     double gravity_constant = 9.8;
     std::array<double, 4> imu_calibration{{1., 0., 0., 0.}};
-    common::optional<transform::Rigid3d> fixed_frame_origin_in_map;
+    absl::optional<transform::Rigid3d> fixed_frame_origin_in_map;
   };
 
   enum class TrajectoryState { ACTIVE, FINISHED, FROZEN, DELETED };

diff --git a/cartographer/mapping/trajectory_node.h b/cartographer/mapping/trajectory_node.h
@@ -21,7 +21,7 @@
 #include <vector>
 
 #include "Eigen/Core"
-#include "cartographer/common/optional.h"
+#include "absl/types/optional.h"
 #include "cartographer/common/time.h"
 #include "cartographer/mapping/proto/trajectory_node_data.pb.h"
 #include "cartographer/sensor/range_data.h"
@@ -38,7 +38,7 @@ struct TrajectoryNodePose {
   // The node pose in the global SLAM frame.
   transform::Rigid3d global_pose;
 
-  common::optional<ConstantPoseData> constant_pose_data;
+  absl::optional<ConstantPoseData> constant_pose_data;
 };
 
 struct TrajectoryNode {

diff --git a/cartographer/sensor/collator_interface.h b/cartographer/sensor/collator_interface.h
@@ -22,7 +22,7 @@
 #include <unordered_set>
 #include <vector>
 
-#include "cartographer/common/optional.h"
+#include "absl/types/optional.h"
 #include "cartographer/sensor/data.h"
 
 namespace cartographer {
@@ -61,7 +61,7 @@ class CollatorInterface {
   // the ID of the trajectory that needs more data before CollatorInterface is
   // unblocked. Returns 'nullopt' for implementations that do not wait for a
   // particular trajectory.
-  virtual common::optional<int> GetBlockingTrajectoryId() const = 0;
+  virtual absl::optional<int> GetBlockingTrajectoryId() const = 0;
 };
 
 }  // namespace sensor

diff --git a/cartographer/sensor/fixed_frame_pose_data.cc b/cartographer/sensor/fixed_frame_pose_data.cc
@@ -16,7 +16,7 @@
 
 #include "cartographer/sensor/fixed_frame_pose_data.h"
 
-#include "cartographer/common/optional.h"
+#include "absl/types/optional.h"
 #include "cartographer/transform/transform.h"
 
 namespace cartographer {
@@ -34,9 +34,9 @@ proto::FixedFramePoseData ToProto(const FixedFramePoseData& pose_data) {
 FixedFramePoseData FromProto(const proto::FixedFramePoseData& proto) {
   return FixedFramePoseData{common::FromUniversal(proto.timestamp()),
                             proto.has_pose()
-                                ? common::optional<transform::Rigid3d>(
+                                ? absl::optional<transform::Rigid3d>(
                                       transform::ToRigid3(proto.pose()))
-                                : common::optional<transform::Rigid3d>()};
+                                : absl::optional<transform::Rigid3d>()};
 }
 
 }  // namespace sensor

diff --git a/cartographer/sensor/fixed_frame_pose_data.h b/cartographer/sensor/fixed_frame_pose_data.h
@@ -19,7 +19,7 @@
 
 #include <memory>
 
-#include "cartographer/common/optional.h"
+#include "absl/types/optional.h"
 #include "cartographer/common/time.h"
 #include "cartographer/sensor/proto/sensor.pb.h"
 #include "cartographer/transform/rigid_transform.h"
@@ -31,7 +31,7 @@ namespace sensor {
 // optimization.
 struct FixedFramePoseData {
   common::Time time;
-  common::optional<transform::Rigid3d> pose;
+  absl::optional<transform::Rigid3d> pose;
 };
 
 // Converts 'pose_data' to a proto::FixedFramePoseData.

diff --git a/cartographer/sensor/internal/collator.cc b/cartographer/sensor/internal/collator.cc
@@ -47,8 +47,8 @@ void Collator::AddSensorData(const int trajectory_id,
 
 void Collator::Flush() { queue_.Flush(); }
 
-common::optional<int> Collator::GetBlockingTrajectoryId() const {
-  return common::optional<int>(queue_.GetBlocker().trajectory_id);
+absl::optional<int> Collator::GetBlockingTrajectoryId() const {
+  return absl::optional<int>(queue_.GetBlocker().trajectory_id);
 }
 
 }  // namespace sensor