commit 1b8a4d5d11671ed83cf6077e363dd95333f08ef8
author Sameer Agarwal <sameeragarwal@google.com> Fri Aug 17 16:49:11 2012 -0700
committer ceres-solver code review <noreply-gerritcodereview@google.com> Fri Aug 17 16:56:10 2012 -0700
tree 53ccf5c88679e9d377412f3e88e8021e352f1267
parent 31c1e784ab2cb9294c6e05414cf06aae2b3766de

Fix a minor bug in detect_structure logging.

Change-Id: I117f7745e4c67595b3ff9244cde82b5b5b34ee4b

internal/ceres/detect_structure.cc

diff --git a/internal/ceres/detect_structure.cc b/internal/ceres/detect_structure.cc
index 90ee532..ea5bf2e 100644
--- a/internal/ceres/detect_structure.cc
+++ b/internal/ceres/detect_structure.cc
@@ -60,10 +60,10 @@
       *row_block_size = row.block.size;
     } else if (*row_block_size != Eigen::Dynamic &&
                *row_block_size != row.block.size) {
-      *row_block_size = Eigen::Dynamic;
       VLOG(2) << "Dynamic row block size because the block size changed from "
               << *row_block_size << " to "
               << row.block.size;
+      *row_block_size = Eigen::Dynamic;
     }
 
 
@@ -71,10 +71,10 @@
       *e_block_size = bs.cols[e_block_id].size;
     } else if (*e_block_size != Eigen::Dynamic &&
                *e_block_size != bs.cols[e_block_id].size) {
-      *e_block_size = Eigen::Dynamic;
       VLOG(2) << "Dynamic e block size because the block size changed from "
               << *e_block_size << " to "
               << bs.cols[e_block_id].size;
+      *e_block_size = Eigen::Dynamic;
     }
 
     if (*f_block_size == 0) {
@@ -85,11 +85,11 @@
     } else if (*f_block_size != Eigen::Dynamic) {
       for (int c = 1; c < row.cells.size(); ++c) {
         if (*f_block_size != bs.cols[row.cells[c].block_id].size) {
-          *f_block_size = Eigen::Dynamic;
           VLOG(2) << "Dynamic f block size because the block size "
-                    << "changed from " << *f_block_size << " to "
-                    << bs.cols[row.cells[c].block_id].size;
-            break;
+                  << "changed from " << *f_block_size << " to "
+                  << bs.cols[row.cells[c].block_id].size;
+          *f_block_size = Eigen::Dynamic;
+          break;
         }
       }
     }