Re: [PATCH v3 0/1] MR5236: ntdll: Implement NtdllDefWindowProc_A().

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/misc.c:
> -
>
> +/******************************************************************************
> -
> -
>              NtdllDefWindowProc_A (NTDLL.@)
> - */
>
> +LRESULT WINAPI NtdllDefWindowProc_A( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam )
> +{
> - LPARAM (WINAPI *pDefWindowProcA)(HWND,UINT,WPARAM,LPARAM); /* DefWindowProcA */
> -
> - const UNICODE_STRING name = RTL_CONSTANT_STRING( L"user32.dll" );
> - NTSTATUS status;
> - HMODULE module;
> -
> - if (( status = LdrGetDllHandle( NULL, 0, &name, &module )))
> - {
> -
>    ERR("Failed to get user32.dll handle, status %ld\n", status);
> -
>    return 1; /* FIXME: Is this a good way to indicate failure? */

DefWindowProcA seems to return 0 on failure.

```suggestion:-0+0
        return 0; /* FIXME: Is this a good way to indicate failure? */
```