Lint cleanup from Jim Roseborough. Change-Id: I6ddbf5c3d66595d27f7967a309768e5f5dd7e1fd
diff --git a/internal/ceres/corrector.cc b/internal/ceres/corrector.cc index cbf49ec..f9deb87 100644 --- a/internal/ceres/corrector.cc +++ b/internal/ceres/corrector.cc
@@ -85,7 +85,7 @@ } // We now require that the first derivative of the loss function be - // positive only if the second derivative is non-zero. This is + // positive only if the second derivative is positive. This is // because when the second derivative is non-positive, we do not use // the second order correction suggested by BANS and instead use a // simpler first order strategy which does not use a division by the
diff --git a/internal/ceres/covariance_impl.cc b/internal/ceres/covariance_impl.cc index c302181..6fe7a80 100644 --- a/internal/ceres/covariance_impl.cc +++ b/internal/ceres/covariance_impl.cc
@@ -447,7 +447,8 @@ return false; } - LinearSolverTerminationType termination_type = ss.Cholesky(&cholmod_jacobian_view, factor); + LinearSolverTerminationType termination_type = + ss.Cholesky(&cholmod_jacobian_view, factor); event_logger.AddEvent("Numeric Factorization"); if (termination_type != TOLERANCE) {
diff --git a/internal/ceres/levenberg_marquardt_strategy.cc b/internal/ceres/levenberg_marquardt_strategy.cc index 009a951..2227da6 100644 --- a/internal/ceres/levenberg_marquardt_strategy.cc +++ b/internal/ceres/levenberg_marquardt_strategy.cc
@@ -108,8 +108,8 @@ if (linear_solver_summary.termination_type == FATAL_ERROR) { LOG(WARNING) << "Linear solver fatal error."; - } else if (linear_solver_summary.termination_type == FAILURE || - !IsArrayValid(num_parameters, step)) { + } else if (linear_solver_summary.termination_type == FAILURE || + !IsArrayValid(num_parameters, step)) { LOG(WARNING) << "Linear solver failure. Failed to compute a finite step."; linear_solver_summary.termination_type = FAILURE; } else {