Skip to content

Commit

Permalink
Replace some instances of std::size_t with size_t
Browse files Browse the repository at this point in the history
  • Loading branch information
nvdbaranec committed Jul 20, 2021
1 parent 7ced768 commit b237724
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions cpp/src/copying/concatenate.cu
Original file line number Diff line number Diff line change
Expand Up @@ -358,9 +358,8 @@ class traverse_children {
[](size_t a, auto const& b) -> size_t { return a + b.size(); }) +
1;
// note: output text must include "exceeds size_type range" for python error handling
CUDF_EXPECTS(
total_offset_count <= static_cast<std::size_t>(std::numeric_limits<size_type>::max()),
"Total number of concatenated offsets exceeds size_type range");
CUDF_EXPECTS(total_offset_count <= static_cast<size_t>(std::numeric_limits<size_type>::max()),
"Total number of concatenated offsets exceeds size_type range");
}
};

Expand All @@ -382,7 +381,7 @@ void traverse_children::operator()<cudf::string_view>(host_span<column_view cons
cudf::detail::get_value<offset_type>(scv.offsets(), scv.offset(), stream));
});
// note: output text must include "exceeds size_type range" for python error handling
CUDF_EXPECTS(total_char_count <= static_cast<std::size_t>(std::numeric_limits<size_type>::max()),
CUDF_EXPECTS(total_char_count <= static_cast<size_t>(std::numeric_limits<size_type>::max()),
"Total number of concatenated chars exceeds size_type range");
}

Expand Down Expand Up @@ -453,7 +452,7 @@ void bounds_and_type_check(host_span<column_view const> cols, rmm::cuda_stream_v
return a + static_cast<size_t>(b.size());
});
// note: output text must include "exceeds size_type range" for python error handling
CUDF_EXPECTS(total_row_count <= static_cast<std::size_t>(std::numeric_limits<size_type>::max()),
CUDF_EXPECTS(total_row_count <= static_cast<size_t>(std::numeric_limits<size_type>::max()),
"Total number of concatenated rows exceeds size_type range");

// traverse children
Expand Down

0 comments on commit b237724

Please sign in to comment.