Fix minor compiler warnings

src/theia/sfm/bundle_adjustment/bundle_adjust_two_views.cc

@@ -61,7 +61,6 @@ void SetSolverOptions(const BundleAdjustmentOptions& options,
   solver_options->visibility_clustering_type = ceres::CANONICAL_VIEWS;
   solver_options->logging_type = ceres::SILENT;
   solver_options->num_threads = options.num_threads;
-  solver_options->num_linear_solver_threads = options.num_threads;
   solver_options->max_num_iterations = 200;
   // Solver options takes ownership of the ordering so that we can order the BA
   // problem by points and cameras.

src/theia/sfm/bundle_adjustment/bundle_adjuster.cc

@@ -63,7 +63,6 @@ void SetSolverOptions(const BundleAdjustmentOptions& options,
   solver_options->logging_type =
       options.verbose ? ceres::PER_MINIMIZER_ITERATION : ceres::SILENT;
   solver_options->num_threads = options.num_threads;
-  solver_options->num_linear_solver_threads = options.num_threads;
   solver_options->max_num_iterations = options.max_num_iterations;
   solver_options->max_solver_time_in_seconds =
       options.max_solver_time_in_seconds;

src/theia/sfm/feature_extractor.cc

@@ -156,6 +156,7 @@ bool FeatureExtractor::ExtractFeatures(
     keypoints->clear();
     descriptors->clear();
   }
+  return true;
 }
 
 bool FeatureExtractor::ExtractFeaturesFromImage(

src/theia/sfm/global_pose_estimation/nonlinear_position_estimator.cc

@@ -34,10 +34,10 @@
 
 #include "theia/sfm/global_pose_estimation/nonlinear_position_estimator.h"
 
-#include <ceres/ceres.h>
-#include <ceres/rotation.h>
 #include <Eigen/Core>
 #include <algorithm>
+#include <ceres/ceres.h>
+#include <ceres/rotation.h>
 #include <unordered_map>
 #include <unordered_set>
 #include <utility>
@@ -134,7 +134,6 @@ bool NonlinearPositionEstimator::EstimatePositions(
   // Set the solver options.
   ceres::Solver::Summary summary;
   solver_options_.num_threads = options_.num_threads;
-  solver_options_.num_linear_solver_threads = options_.num_threads;
   solver_options_.max_num_iterations = options_.max_num_iterations;
 
   // Choose the type of linear solver. For sufficiently large problems, we want
@@ -208,9 +207,10 @@ void NonlinearPositionEstimator::AddCameraToCameraConstraints(
                                position2->data());
   }
 
-  VLOG(2) << problem_->NumResidualBlocks() << " camera to camera constraints "
-                                              "were added to the position "
-                                              "estimation problem.";
+  VLOG(2) << problem_->NumResidualBlocks()
+          << " camera to camera constraints "
+             "were added to the position "
+             "estimation problem.";
 }
 
 void NonlinearPositionEstimator::AddPointToCameraConstraints(
@@ -233,15 +233,14 @@ void NonlinearPositionEstimator::AddPointToCameraConstraints(
   for (const TrackId track_id : tracks_to_add) {
     triangulated_points_[track_id] = 100.0 * rng_->RandVector3d();
 
-    AddTrackToProblem(track_id,
-                      orientations,
-                      point_to_camera_weight,
-                      positions);
+    AddTrackToProblem(
+        track_id, orientations, point_to_camera_weight, positions);
   }
 
-  VLOG(2) << num_point_to_camera_constraints << " point to camera constriants "
-                                                "were added to the position "
-                                                "estimation problem.";
+  VLOG(2) << num_point_to_camera_constraints
+          << " point to camera constriants "
+             "were added to the position "
+             "estimation problem.";
 }
 
 int NonlinearPositionEstimator::FindTracksForProblem(
@@ -269,8 +268,8 @@ int NonlinearPositionEstimator::FindTracksForProblem(
         GetTracksSortedByNumViews(reconstruction_, *view, *tracks_to_add);
 
     for (int i = 0;
-         i < sorted_tracks.size() && tracks_per_camera[position.first] <
-                                         options_.min_num_points_per_view;
+         i < sorted_tracks.size() &&
+         tracks_per_camera[position.first] < options_.min_num_points_per_view;
          i++) {
       // Update the number of point to camera constraints for each camera.
       tracks_to_add->insert(sorted_tracks[i]);
@@ -320,7 +319,8 @@ std::vector<TrackId> NonlinearPositionEstimator::GetTracksSortedByNumViews(
                options_.min_num_points_per_view);
   std::partial_sort(views_per_track.begin(),
                     views_per_track.begin() + num_tracks_to_sort,
-                    views_per_track.end(), CompareViewsPerTrack);
+                    views_per_track.end(),
+                    CompareViewsPerTrack);
 
   for (int i = 0; i < num_tracks_to_sort; i++) {
     sorted_tracks[i] = views_per_track[i].first;

src/theia/sfm/global_pose_estimation/position_estimator.h

@@ -53,6 +53,7 @@ namespace theia {
 class PositionEstimator {
  public:
   PositionEstimator() {}
+  virtual ~PositionEstimator() {}
 
   // Input the view pairs containing relative poses between matched
   // geometrically verified views, as well as the global (absolute) orientations

src/theia/sfm/global_pose_estimation/rotation_estimator.h

@@ -50,7 +50,7 @@ namespace theia {
 class RotationEstimator {
  public:
   RotationEstimator() {}
-
+  virtual ~RotationEstimator() {}
   // Input the view pairs containing relative rotations between matched
   // geometrically verified views and outputs a rotation estimate for each view.
   //

src/theia/sfm/hybrid_reconstruction_estimator.cc

@@ -444,6 +444,7 @@ bool HybridReconstructionEstimator::InitializeCamerasFromTwoViewInfo(
 
   view1->SetEstimated(true);
   view2->SetEstimated(true);
+  return true;
 }
 
 bool HybridReconstructionEstimator::InitializeCamerasWithKnownOrientation(

src/theia/solvers/random_sampler.h

@@ -54,7 +54,7 @@ class RandomSampler : public Sampler {
                 const int min_num_samples);
   ~RandomSampler() {}
 
-  bool Initialize(const int num_datapoints);
+  bool Initialize(const int num_datapoints) override;
 
   // Samples the input variable data and fills the vector subset with the
   // random samples.