[mlpack-git] master: Fix build warnings. (dd83d57)

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Wed Dec 23 11:42:52 EST 2015


Repository : https://github.com/mlpack/mlpack

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/de9cc4b05069e1fa4793d9355f2f595af5ff45d2...6070527af14296cd99739de6c62666cc5d2a2125

>---------------------------------------------------------------

commit dd83d5716133674343f3c69ca25b5fa43b257e6f
Author: ryan <ryan at ratml.org>
Date:   Thu Sep 24 17:30:47 2015 -0400

    Fix build warnings.


>---------------------------------------------------------------

dd83d5716133674343f3c69ca25b5fa43b257e6f
 src/mlpack/methods/hoeffding_trees/hoeffding_split_impl.hpp | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/mlpack/methods/hoeffding_trees/hoeffding_split_impl.hpp b/src/mlpack/methods/hoeffding_trees/hoeffding_split_impl.hpp
index 2b536f3..7303874 100644
--- a/src/mlpack/methods/hoeffding_trees/hoeffding_split_impl.hpp
+++ b/src/mlpack/methods/hoeffding_trees/hoeffding_split_impl.hpp
@@ -27,8 +27,8 @@ HoeffdingSplit<
     datasetInfo(datasetInfo),
     successProbability(successProbability),
     splitDimension(size_t(-1)),
-    numericSplit(),
-    categoricalSplit(0)
+    categoricalSplit(0),
+    numericSplit()
 {
   for (size_t i = 0; i < dimensionality; ++i)
   {
@@ -158,6 +158,8 @@ size_t HoeffdingSplit<
     return numericSplit.CalculateDirection(point[splitDimension]);
   else if (datasetInfo.Type(splitDimension) == data::Datatype::categorical)
     return categoricalSplit.CalculateDirection(point[splitDimension]);
+  else
+    return 0; // Not sure what to do here...
 }
 
 template<
@@ -170,7 +172,7 @@ size_t HoeffdingSplit<
     FitnessFunction,
     NumericSplitType,
     CategoricalSplitType
->::Classify(const VecType& point) const
+>::Classify(const VecType& /* point */) const
 {
   // We're a leaf (or being considered a leaf), so classify based on what we
   // know.



More information about the mlpack-git mailing list