edponce commented on a change in pull request #10274:
URL: https://github.com/apache/arrow/pull/10274#discussion_r632247362
##########
File path: cpp/src/arrow/util/int_util_internal.h
##########
@@ -63,9 +63,9 @@ OPS_WITH_OVERFLOW(DivideWithOverflow, div)
#undef OP_WITH_OVERFLOW
#undef OPS_WITH_OVERFLOW
-// Define functions NegateWithOverflow with the signature `bool(T u, T* out)`
-// where T is a signed integer type. On overflow, these functions return true.
-// Otherwise, false is returned and `out` is updated with the result of the
+// Define functions NegateWithOverflow with the signature
+// `bool(T u, T* out)` where T is a signed integer type. On overflow, these
functions
+// return true. Otherwise, false is returned and `out` is updated with the
result of the
Review comment:
It was a spurious change because I had initially added an
`AbsoluteValueWithOverflow` but `NegateWithOverflow` sufficed.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
