[ARROW-13444] [C++] C++20 compatibility by updating std::result_of to std::invoke_result

• Type: Wish
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.1
• Fix Version/s: 6.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/29110
• Assignee: Ben Kietzman
• Reporter: Anders Wind
• Created: 26/Jul/21 07:56
• Updated: 11/Jan/23 08:33
• Resolved: 27/Jul/21 19:57

Description

There are currently 5 header files which uses std::result_of. As specified here https://en.cppreference.com/w/cpp/types/result_of, std::result_of got deprecated in c+17 and removed in c20. Because of this, its not possible to include the c+ arrow headers in a project using c++20 without getting compilation errors.

To solve this, cppreference specifies how std::invoke_result can be used instead of std::result_of. Unfortunately std::invoke_result was only added in c+17 and therefore I expect that solving this will require some macro work, to keep c+14 and lower support.

 

This is  the first time I post an issue and I hope I have followed the rules.

Issue Links

• links to: GitHub Pull Request #10814