Fix undefined symbols in vpLuminanceDCT

lagadic · May 23, 2024 · a235ee2 · a235ee2
1 parent c8e76e3
commit a235ee2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/modules/visual_features/include/visp3/visual_features/vpFeatureLuminanceMapping.h b/modules/visual_features/include/visp3/visual_features/vpFeatureLuminanceMapping.h
@@ -275,7 +275,7 @@ class VISP_EXPORT vpLuminanceDCT : public vpLuminanceMapping
    * @brief Helper class to iterate and get/set the values from a matrix, following a zigzag pattern.
    *
    */
-  class VISP_EXPORT vpMatrixZigZagIndex
+  class vpMatrixZigZagIndex
   {
   public:
     vpMatrixZigZagIndex();
@@ -289,7 +289,6 @@ class VISP_EXPORT vpLuminanceDCT : public vpLuminanceMapping
     /**
      * @brief Fill the vector s with (end - start) values, according to the zigzag matrix indexing strategy
      *
-     *
      * @param m the matrix
      * @param start The first value. Use 0 to start with the matrix's top left value
      * @param end The last value to store in the vector. (exclusive)
@@ -340,7 +339,7 @@ class VISP_EXPORT vpLuminanceDCT : public vpLuminanceMapping
   vpLuminanceDCT(const vpLuminanceDCT &other);
 
 
-  vpLuminanceDCT &operator=(const vpLuminanceDCT &other);
+  vpLuminanceDCT &operator=(const vpLuminanceDCT &other) = default;
 
 
   void map(const vpImage<unsigned char> &I, vpColVector &s) vp_override;

diff --git a/modules/visual_features/src/visual-feature/vpFeatureLuminanceMapping.cpp b/modules/visual_features/src/visual-feature/vpFeatureLuminanceMapping.cpp
@@ -390,6 +390,10 @@ void vpLuminanceDCT::vpMatrixZigZagIndex::setValues(const vpColVector &s, unsign
 
 // vpLuminanceDCT
 
+vpLuminanceDCT::vpLuminanceDCT(const vpLuminanceDCT &other) : vpLuminanceMapping(other.getProjectionSize())
+{
+  *this = other;
+}
 
 void vpLuminanceDCT::map(const vpImage<unsigned char> &I, vpColVector &s)
 {
@@ -404,8 +408,6 @@ void vpLuminanceDCT::map(const vpImage<unsigned char> &I, vpColVector &s)
   m_zigzag.getValues(m_dct, 0, m_mappingSize, s);
 }
 
-
-
 void vpLuminanceDCT::computeDCTMatrix(vpMatrix &D, unsigned int n) const
 {
   D.resize(n, n, false, false);