Remove redundant code

Author: zhujun98

include/xtensor/xhistogram.hpp

@@ -76,7 +76,7 @@ namespace xt
                     // left and right are not bounds of data
                     if ( v >= left & v < right )
                     {
-                        auto i_bin = static_cast<size_t>(static_cast<double>(n_bins) * (static_cast<double>(data(i)) - left) * norm);
+                        auto i_bin = static_cast<size_t>(static_cast<double>(n_bins) * (v - left) * norm);
                         count(i_bin) += weights(i);
                     }
                     else if ( v == right )

test/test_xhistogram.cpp

@@ -14,8 +14,6 @@
 #include "xtensor/xtensor.hpp"
 #include "xtensor/xhistogram.hpp"
 #include "xtensor/xrandom.hpp"
-#include "xtensor/xio.hpp"
-
 
 namespace xt
 {
@@ -31,16 +29,6 @@ namespace xt
             EXPECT_EQ(count(1), 2.);
         }
 
-        {
-            xt::xtensor<double, 1> edges {0, 1.5, 3};
-            xt::xtensor<double, 1> weights {1, 1, 1, 1};
-            auto count = xt::histogram<int>(data, edges, weights, true);
-            // This is indeed a bug. If the count is the density, an integral return
-            // type should not be allowed.
-            EXPECT_EQ(count(0), 0.);
-            EXPECT_EQ(count(1), 0.);
-        }
-
         {
             xt::xtensor<double, 1> count = xt::histogram(data,
                 xt::histogram_bin_edges(data, std::size_t(2), xt::histogram_algorithm::uniform)