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? */
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5236#note_65647