From e1385cc7e7f8fcd952ea0119e349f28c83f3d2aa Mon Sep 17 00:00:00 2001
From: Sergiu Deitsch <sergiu.deitsch@gmail.com>
Date: Mon, 16 Sep 2024 23:57:03 +0200
Subject: [PATCH] Fix MSVC build

Previously missed instance of preprocessor directives used in a macro
expansion.

Change-Id: I2f1e4ad95036851fa502a9ea01d2a9684a3e0f9d
---
 internal/ceres/sparse_cholesky_test.cc | 30 +++++++++++++-------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/internal/ceres/sparse_cholesky_test.cc b/internal/ceres/sparse_cholesky_test.cc
index 931c93df6..4142a4caf 100644
--- a/internal/ceres/sparse_cholesky_test.cc
+++ b/internal/ceres/sparse_cholesky_test.cc
@@ -224,25 +224,25 @@ TEST_P(SparseCholeskyTest, FactorAndSolve) {
 namespace {
 
 #ifndef CERES_NO_SUITESPARSE
-INSTANTIATE_TEST_SUITE_P(
-    SuiteSparseCholesky,
-    SparseCholeskyTest,
-    ::testing::Combine(::testing::Values(SUITE_SPARSE),
+const auto SuiteSparseCholeskyParameters = ::testing::Combine(
+    ::testing::Values(SUITE_SPARSE),
 #if defined(CERES_NO_CHOLMOD_FLOAT)
-                       ::testing::Values(false),
+    ::testing::Values(false),
 #else
-                       ::testing::Values(false, true),
+    ::testing::Values(false, true),
 #endif  // defined(CERES_NO_CHOLMOD_FLOAT)
 #if defined(CERES_NO_CHOLMOD_PARTITION)
-                       ::testing::Values(OrderingType::AMD,
-                                         OrderingType::NATURAL),
+    ::testing::Values(OrderingType::AMD, OrderingType::NATURAL),
 #else
-                       ::testing::Values(OrderingType::AMD,
-                                         OrderingType::NESDIS,
-                                         OrderingType::NATURAL),
+    ::testing::Values(
+        OrderingType::AMD, OrderingType::NESDIS, OrderingType::NATURAL),
 #endif  // defined(CERES_NO_CHOLMOD_PARTITION)
-                       ::testing::Values(true, false)),
-    ParamInfoToString);
+    ::testing::Values(true, false));
+
+INSTANTIATE_TEST_SUITE_P(SuiteSparseCholesky,
+                         SparseCholeskyTest,
+                         SuiteSparseCholeskyParameters,
+                         ParamInfoToString);
 #endif  // !defined(CERES_NO_SUITESPARSE)
 
 #ifndef CERES_NO_ACCELERATE_SPARSE
@@ -259,7 +259,7 @@ INSTANTIATE_TEST_SUITE_P(
 #endif
 
 #ifdef CERES_USE_EIGEN_SPARSE
-const auto Parameters = ::testing::Combine(
+const auto EigenSparseCholeskyParameters = ::testing::Combine(
     ::testing::Values(EIGEN_SPARSE),
     ::testing::Values(false, true),
 #if defined(CERES_NO_EIGEN_METIS)
@@ -272,7 +272,7 @@ const auto Parameters = ::testing::Combine(
 
 INSTANTIATE_TEST_SUITE_P(EigenSparseCholesky,
                          SparseCholeskyTest,
-                         Parameters,
+                         EigenSparseCholeskyParameters,
                          ParamInfoToString);
 #endif  // CERES_USE_EIGEN_SPARSE