Re: [PATCH v2 0/4] MR8515: setupapi: Support in-built properties for device instances in SetupDiGetDevicePropertyW.

[Connor McAdams (＠cmcadams)]

Connor McAdams (@cmcadams) commented about dlls/setupapi/devinst.c:
> +static DWORD get_device_property( struct device *device, const DEVPROPKEY *prop_key, DEVPROPTYPE *prop_type,
> -
>                              BYTE *buf, DWORD buf_size, DWORD *req_size, DWORD flags )
>
> +{
> - DWORD ret = ERROR_SUCCESS;
> -
> - if (!prop_key)
> -
>    return ERROR_INVALID_DATA;
> - if (!prop_type || (!buf && buf_size))
> -
>    return ERROR_INVALID_USER_BUFFER;
> - if (flags)
> -
>    return ERROR_INVALID_FLAGS;
> -
> - if (IsEqualDevPropKey( *prop_key, DEVPKEY_Device_InstanceId ))
> - {
> -
>    DWORD size = (wcslen( device->instanceId ) + 1) * sizeof( WCHAR );

Newline after this to separate variable declaration/code. Same goes for the `else if` below.