Re: [PATCH 2/4] msvcr110: Add _Context::_CurrentContext.

[Piotr Caban]

Hi Daniel,

On 11/2/20 5:21 AM, Daniel Lehman wrote:
> +    if (0) /* crash Windows */
> +        p__Context__CurrentContext(NULL);
> It's not possible to call it with NULL argument using proper 
> compiler/headers. Please remove the test.
> +#if _MSVCR_VER > 100
> +/* ?_CurrentContext(a)_Context@details(a)Concurrency@@SA?AV123(a)XZ */
> +_Context *__cdecl _Context__CurrentContext(_Context *ctx)
> Could you please rename the ctx argument to ret? I was trying to always 
> name it this way to mark structures returned in memory.

Thanks,
Piotr