Merge pull request #1280 from MRPT/fix/cs-external-linking

Fix C linkage build error in CSparse

Author: jlblancoc

doc/source/doxygen-docs/changelog.md

@@ -1,7 +1,8 @@
 \page changelog Change Log
 
 # Version 2.10.2: UNRELEASED
-(No changes yet)
+- BUG FIXES:
+  - Fix CSparse "C" linkage build error (OSX Clang)
 
 # Version 2.10.1: Released August 10th, 2023
 - Build system:

libs/math/include/mrpt/math/CSparseMatrix.h

@@ -19,18 +19,14 @@
 #include <stdexcept>
 
 // Include CSparse lib headers, either from the system or embedded:
-extern "C"
-{
 #if MRPT_HAS_SUITESPARSE
-#define NCOMPLEX  // In MRPT we don't need complex numbers, so avoid the
-// annoying warning: 'cs_ci_house' has C-linkage specified,
-// but returns UDT 'std::complex<double>' which is
-// incompatible with C
 #include "cs.h"
 #else
+extern "C"
+{
 #include <mrpt/3rdparty/CSparse/cs.h>
-#endif
 }
+#endif
 
 namespace mrpt::math
 {