You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/01/11 16:54:28 UTC

[GitHub] [arrow] westonpace commented on a change in pull request #9095: ARROW-10183: [C++] Apply composable futures to CSV

westonpace commented on a change in pull request #9095:
URL: https://github.com/apache/arrow/pull/9095#discussion_r555194510



##########
File path: cpp/src/arrow/util/iterator.h
##########
@@ -35,6 +36,13 @@
 
 namespace arrow {
 
+namespace detail {
+
+template <typename Signature>
+using result_of_t = typename std::result_of<Signature>::type;

Review comment:
       It was unused in iterator.h so I just removed it.  I think I pulled it in because I was using Loop as a base for the transforming iterator but it mutated enough to longer require this.




----------------------------------------------------------------
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:
users@infra.apache.org