Re: [PATCH 0/4] MR1157: win32u: Move DisplayConfigGetDeviceInfo from user32 and implement DISPLAYCONFIG_DEVICE_INFO_GET_TARGET_NAME.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
> -
> -
>    memset( &target_name->flags, 0, sizeof(*target_name) - offsetof(DISPLAYCONFIG_TARGET_DEVICE_NAME, flags) );
> -
> -
>    LIST_FOR_EACH_ENTRY(monitor, &monitors, struct monitor, entry)
> -
>    {
> -
>        if (target_name->header.id != monitor->output_id) continue;
> -
>        if (memcmp( &target_name->header.adapterId, &monitor->adapter->gpu_luid,
> -
>                    sizeof(monitor->adapter->gpu_luid) ))
> -
>            continue;
> -
> -
>        /* FIXME: get real monitor name. */
> -
>        snprintf( buffer, ARRAY_SIZE(buffer), "Display%u", monitor->output_id + 1 );
> -
>        asciiz_to_unicode( target_name->monitorFriendlyDeviceName, buffer );
> -
>        target_name->flags.friendlyNameForced = 1;
> -
>        lstrcpyW( target_name->monitorDevicePath, monitor->dev.interface_name );
> -
>        ret = ERROR_SUCCESS;

You should use STATUS_SUCCESS instead ERROR_SUCCESS.