Changed result_of to invoke_result for C++20 compatibility.

[mkloczko]

Result_of is no longer in C++20.

[mkloczko]

Oh well it did work locally.

[emil-e]

Oh well it did work locally.

That's because invoke_result is not available in C++11 and CI actually builds for C++11.

[mkloczko]

Maybe a class could be implemented that chooses either result_of or invoke_result based on C++ version using ifdefs?

[emil-e]

That would be an appropriate solution I think.

…

On Sat, 28 Nov 2020 at 16:32 Mateusz Kłoczko ***@***.***> wrote: Maybe a class could be implemented that chooses either result_of or invoke_result based on C++ version using ifdefs? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#261 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABLY3DXCKXW6URXZR2WDOJ3SSEJXLANCNFSM4UFXMUDQ> .

[mkloczko]

Where could I add such a class?

[Warwolt]

@mkloczko One alternative would be to replace all the usages of std::result_of with a macro or intermediate template and use an #ifdef inside that intermediary to select between passing the arguments to std::result_of or std::invoke_result based on the selected compiler standard.

[emil-e]

@Warwolt I don't have time to do this myself but what I would do is create a class (struct) template that inherits from the appropriate one using ifdefs.

[mkloczko]

Just in case the newer solution is in #269 (couldn't change the source branch).