commit 4a6cc1c410d963274d67a95cf0fac3db51e5bc92
author Sameer Agarwal <sameeragarwal@google.com> Mon Jun 18 10:20:08 2012 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Mon Jun 18 11:17:52 2012 -0700
tree 16fda2a8f16e30d283f3b56e8bbc619df87c57be
parent eb893404faa2bb35705277829714e6ba80ae9484

Various fixes repored by Markus Moll.

1. Termination type was using == instead of =.
2. LevenbergMarquardtStrategyTest was using an object
   which was not returning the correct value.
3. DumpLinearLeastSquaresProblemToTextFile was passing an
   unnecessary argument to StringAppendF.

Change-Id: Ie7598c8e3d504763c889737a0d5cec805d52bbaf

internal/ceres/solver_impl.cc

diff --git a/internal/ceres/solver_impl.cc b/internal/ceres/solver_impl.cc
index ab91681..982fb40 100644
--- a/internal/ceres/solver_impl.cc
+++ b/internal/ceres/solver_impl.cc
@@ -216,7 +216,7 @@
   if (!program->CopyUserStateToParameterBlocks())  {
     // This can only happen if there was a numerical problem updating the local
     // jacobians. Indicate as such and fail out.
-    summary->termination_type == NUMERICAL_FAILURE;
+    summary->termination_type = NUMERICAL_FAILURE;
     summary->error = "Local parameterization failure.";
     return;
   }