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 {
