commit aa62dd86a8cd978cd75245a711ef3bbdd0defb15
author Sameer Agarwal <sameeragarwal@google.com> Thu May 19 11:19:52 2022 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Thu May 19 11:20:38 2022 -0700
tree 8d343019b56e0eb3e952c54ee257764cf79a8a4e
parent 41c5fb1e807178170f38e7c011922b35d192576b

Fix a build breakage

Change-Id: I57591bc42d53f9856b49f7a16732a2a1e259dc67

internal/ceres/suitesparse.cc

diff --git a/internal/ceres/suitesparse.cc b/internal/ceres/suitesparse.cc
index 82d20d4..8f7a4c7 100644
--- a/internal/ceres/suitesparse.cc
+++ b/internal/ceres/suitesparse.cc
@@ -183,14 +183,13 @@
         StringPrintf("cholmod_analyze failed. error code: %d", cc_.status);
     return nullptr;
   }
-  CHECK(factor != nullptr);
 
+  CHECK(factor != nullptr);
   if (VLOG_IS_ON(2)) {
     cholmod_print_common(const_cast<char*>("Symbolic Analysis"), &cc_);
   }
-}
 
-return factor;
+  return factor;
 }
 
 cholmod_factor* SuiteSparse::AnalyzeCholeskyWithGivenOrdering(

internal/ceres/suitesparse.h

diff --git a/internal/ceres/suitesparse.h b/internal/ceres/suitesparse.h
index e0c485d..14dd2b1 100644
--- a/internal/ceres/suitesparse.h
+++ b/internal/ceres/suitesparse.h
@@ -44,13 +44,12 @@
 #include <vector>
 
 #include "SuiteSparseQR.hpp"
+#include "ceres/internal/disable_warnings.h"
 #include "ceres/linear_solver.h"
 #include "ceres/sparse_cholesky.h"
 #include "cholmod.h"
 #include "glog/logging.h"
 
-#include "ceres/internal/disable_warnings.h"
-
 namespace ceres::internal {
 
 class CompressedRowSparseMatrix;