Define co_await operators for C++20 coroutines - #765

/azp run

Thanks Koby, this looks reasonable but I'd prefer to change the definition in one place rathe than duplicate it. Perhaps you can experiment with updating the following? This needs to support both C++20 and C++17 with the Visual C++ /await flag.

Thanks Koby, this looks reasonable but I'd prefer to change the definition in one place rathe than duplicate it. Perhaps you can experiment with updating the following? This needs to support both C++20 and C++17 with the Visual C++ /await flag.

cppwinrt/strings/base_macros.h

Lines 30 to 32 in 714c085

	#if defined(_RESUMABLE_FUNCTIONS_SUPPORTED) && !defined(__cpp_coroutines)
	#define __cpp_coroutines
	#endif

I assume you mean define a cppwinrt-specific macro here and use it, rather than define the Coroutines TS feature test macro in C++20 coroutines mode too, right? I'll update the PR accordingly.

Perhaps the base_macros.h would look something like this:

#if defined(_RESUMABLE_FUNCTIONS_SUPPORTED) && !defined(__cpp_lib_coroutine) 
#define __cpp_lib_coroutine 
#endif 

And then subsequent checks would simply test for the C++20 feature macro. I think that would work, although I haven't tested it.

Couldn't that mislead other libraries which use the macro for feature detection as well?

Perhaps - then we should define a WINRT_IMPL_COROUTINES macro.

You also need to account for this:

Of course, if we can fix base_macros.h to not define a feature macro and then update the rest, as Koby originally did, to rely only on those feature macros that would save introducing yet another macro.

/azp run

Kenny Kerr (@kennykerr) Any additional comments?

That looks great - I've spun a build. If all looks good I'll merge this in - thanks!