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

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/misc.c:
> -
> - 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? */
> - }
> -
> - pDefWindowProcA = RtlFindExportedRoutineByName( module, "DefWindowProcA" );
> - if (!pDefWindowProcA)
> - {
> -
>    ERR("Failed to load the function\n");
> -
>    return 1;

Ditto. Please duplicate the FIXME here as well.

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