HyukjinKwon commented on code in PR #49113:
URL: https://github.com/apache/arrow/pull/49113#discussion_r2752295603
##########
cpp/src/arrow/tensor_test.cc:
##########
@@ -505,8 +506,16 @@ TYPED_TEST_P(TestFloatTensor, Equals) {
std::vector<int64_t> shape = {4, 4};
- std::vector<c_data_type> c_values = {1, 2, 3, 4, 5, 6, 7, 8,
+ constexpr int c_vals_as_int_arr[] = {1, 2, 3, 4, 5, 6, 7, 8,
Review Comment:
The variable name is a bit lengthty. Intentionally matched the line length
to match with its previous length (so the next line doesn't need the diff). I
can change the variable name :-).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
