boost/histogram
2
0
Fork 0
mirror of https://github.com/boostorg/histogram.git synced 2026-02-21 03:02:11 +00:00
Code Issues Projects Releases Wiki Activity

use std::size_t instead of int for axis indices, use asserts to make sure that arguments for reduce_to are proper, added missing copyright statements

Browse Source
This commit is contained in:
Hans Dembinski
2018-08-14 23:27:42 +02:00
parent a49b81b984
commit 9c15c580a5
19 changed files with 189 additions and 141 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/python/histogram.cpp
View File

@@ -98,7 +98,7 @@ public:
bp::list strides;
auto& b = self.storage_.buffer_;
d["typestr"] = bh::detail::apply(dtype_visitor(), b, shapes, strides);
for (unsigned i = 0; i < self.dim(); ++i) {
for (std::size_t i = 0; i < self.dim(); ++i) {
if (i > 0) strides.append(strides[-1] * shapes[-1]);
shapes.append(self.axis(i).shape());
}
@@ -204,7 +204,7 @@ struct fetcher {
template <typename T>
struct span {
T* data;
unsigned size;
std::size_t size;
const T* begin() const { return data; }
const T* end() const { return data + size; }
};
@@ -297,7 +297,7 @@ bp::object histogram_getitem(const pyhistogram& self, bp::object args) {
}
int idx[BOOST_HISTOGRAM_AXIS_LIMIT];
for (unsigned i = 0; i < dim; ++i) idx[i] = bp::extract<int>(args[i]);
for (std::size_t i = 0; i < dim; ++i) idx[i] = bp::extract<int>(args[i]);
return bp::object(self.at(span<int>{idx, self.dim()}));
}