Remove use of raw pointer from NormalSpaceSampling.

Author: SergioRAgostinho

filters/include/pcl/filters/impl/normal_space.hpp

@@ -59,11 +59,10 @@ pcl::NormalSpaceSampling<PointT, NormalT>::initCompute ()
     return false;
   }
 
-  boost::mt19937 rng (static_cast<unsigned int> (seed_));
-  boost::uniform_int<unsigned int> uniform_distrib (0, unsigned (input_->size ()));
-  delete rng_uniform_distribution_;
-  rng_uniform_distribution_ = new boost::variate_generator<boost::mt19937, boost::uniform_int<unsigned int> > (rng, uniform_distrib);
-
+  rng_uniform_distribution_ = std::bind (
+    boost::uniform_int<unsigned> (0, static_cast<unsigned> (input_->size ())),
+    boost::mt19937 (seed_)
+  );
   return (true);
 }
 
@@ -217,7 +216,7 @@ pcl::NormalSpaceSampling<PointT, NormalT>::applyFilter (std::vector<int> &indice
       // Picking up a sample at random from jth bin
       do
       {
-        random_index = static_cast<unsigned int> ((*rng_uniform_distribution_) () % M);
+        random_index = rng_uniform_distribution_ () % M;
         pos = start_index[j] + random_index;
       } while (is_sampled_flag.test (pos));
 

filters/include/pcl/filters/normal_space.h

@@ -39,8 +39,10 @@
 
 #include <pcl/filters/boost.h>
 #include <pcl/filters/filter_indices.h>
+
 #include <ctime>
 #include <climits>
+#include <functional>
 
 namespace pcl
 {
@@ -77,17 +79,10 @@ namespace pcl
         , binsy_ ()
         , binsz_ ()
         , input_normals_ ()
-        , rng_uniform_distribution_ (nullptr)
       {
         filter_name_ = "NormalSpaceSampling";
       }
 
-      /** \brief Destructor. */
-      ~NormalSpaceSampling ()
-      {
-        delete rng_uniform_distribution_;
-      }
-
       /** \brief Set number of indices to be sampled.
         * \param[in] sample the number of sample indices
         */
@@ -190,7 +185,7 @@ namespace pcl
       isEntireBinSampled (boost::dynamic_bitset<> &array, unsigned int start_index, unsigned int length);
 
       /** \brief Uniform random distribution. */
-      boost::variate_generator<boost::mt19937, boost::uniform_int<std::uint32_t> > *rng_uniform_distribution_;
+      std::function<unsigned ()> rng_uniform_distribution_;
   };
 }