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/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_);