commit cb83b2801e7a62dce6c1cd5431de2becf8aef05f
author Sameer Agarwal <sameeragarwal@google.com> Wed Jun 06 22:26:09 2012 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Wed Jun 06 22:37:33 2012 -0700
tree 5193c7daac52ed04fdabe95127cdb6be404c254a
parent 7a3c43b847a63a2c45d3c58c84081b0a6867f582

Fix broken build.
Fix broken build and verbosity issues.

1. While cleaning up the last CL, I broke a macro.
2. cholmod_common_print was being called too often.

Change-Id: Ia76d8863c72f31b0c02977094b22035ceef835cf

internal/ceres/schur_complement_solver.cc

diff --git a/internal/ceres/schur_complement_solver.cc b/internal/ceres/schur_complement_solver.cc
index 5679a09..f7597e0 100644
--- a/internal/ceres/schur_complement_solver.cc
+++ b/internal/ceres/schur_complement_solver.cc
@@ -287,10 +287,10 @@
     } else {
       factor_ = ss_.AnalyzeCholesky(cholmod_lhs);
     }
-  }
 
-  if (VLOG_IS_ON(2)) {
-    cholmod_print_common("Symbolic Analysis", ss_.mutable_cc());
+    if (VLOG_IS_ON(2)) {
+      cholmod_print_common("Symbolic Analysis", ss_.mutable_cc());
+    }
   }
 
   CHECK_NOTNULL(factor_);

internal/ceres/sparse_normal_cholesky_solver.cc

diff --git a/internal/ceres/sparse_normal_cholesky_solver.cc b/internal/ceres/sparse_normal_cholesky_solver.cc
index 4191f3c..448bbd5 100644
--- a/internal/ceres/sparse_normal_cholesky_solver.cc
+++ b/internal/ceres/sparse_normal_cholesky_solver.cc
@@ -189,10 +189,10 @@
     } else {
       factor_ = ss_.AnalyzeCholesky(lhs.get());
     }
-  }
 
-  if (VLOG_IS_ON(2)) {
-    cholmod_print_common("Symbolic Analysis", ss_.mutable_cc());
+    if (VLOG_IS_ON(2)) {
+      cholmod_print_common("Symbolic Analysis", ss_.mutable_cc());
+    }
   }
 
   CHECK_NOTNULL(factor_);

internal/ceres/suitesparse_test.cc

diff --git a/internal/ceres/suitesparse_test.cc b/internal/ceres/suitesparse_test.cc
index 9211788..0dc418f 100644
--- a/internal/ceres/suitesparse_test.cc
+++ b/internal/ceres/suitesparse_test.cc
@@ -135,7 +135,7 @@
   fill(tsm.mutable_values(), tsm.mutable_values() + 15, 1.0);
   int offset = 0;
 
-#define CERES_TEST_FILL_BLOCK(r, c) \
+#define CERES_TEST_FILL_BLOCK(row_block_id, col_block_id) \
   offset += FillBlock(row_blocks, col_blocks, \
                       row_block_id, col_block_id, \
                       rows + offset, cols + offset);

internal/ceres/visibility_based_preconditioner.cc

diff --git a/internal/ceres/visibility_based_preconditioner.cc b/internal/ceres/visibility_based_preconditioner.cc
index 5a01006..749aa2f 100644
--- a/internal/ceres/visibility_based_preconditioner.cc
+++ b/internal/ceres/visibility_based_preconditioner.cc
@@ -453,10 +453,10 @@
     } else {
       factor_ = ss_.AnalyzeCholesky(lhs);
     }
-  }
 
-  if (VLOG_IS_ON(2)) {
-    cholmod_print_common("Symbolic Analysis", ss_.mutable_cc());
+    if (VLOG_IS_ON(2)) {
+      cholmod_print_common("Symbolic Analysis", ss_.mutable_cc());
+    }
   }
 
   CHECK_NOTNULL(factor_);