Thank you for all these reviews.�� I have submitted v4 to rebase on master and fix whitespace issues.�� I'll address all your issues in v5.

On Thu, Apr 11, 2019 at 3:35 PM Jacek Caban <jacek@codeweavers.com> wrote:

On 4/11/19 8:26 PM, Derek Lesho wrote:
> @@ -3431,7 +3433,8 @@ void WINAPI ExReleaseResourceForThreadLite( PERESOURCE resource, ERESOURCE_THREA
>�� */
>�� void WINAPI KeEnterCriticalRegion(void)
>�� {
> -�� FIXME(": stub\n");
> +�� TRACE(": semi-stub\n");
> +�� KeGetCurrentThread()->critical_region = TRUE;
>�� }

According to MSDN, those may be called recursively, so you'd need a
counter here instead.

Thanks,

Jacek