Re: [PATCH v6 0/9] MR7667: powrprof.dll: Implement functions to enable SteamVR launch

[Alfred Agrell (＠Alcaro)]

Alfred Agrell (@Alcaro) commented about dlls/powrprof/powrprof.c:
> - if (*polguid == NULL)
> - {
> -
>    return ERROR_NOT_ENOUGH_MEMORY;
> - }
> - memcpy(*polguid, active_power_scheme, sizeof(GUID));
> - return ERROR_SUCCESS;
>
> }
>
> DWORD WINAPI PowerSetActiveScheme(HKEY UserRootPowerKey, GUID *polguid)
>  {
> - FIXME("(%p,%s) stub!\n", UserRootPowerKey, wine_dbgstr_guid(polguid));
> - return ERROR_SUCCESS;
> - size_t i;
> -
> - FIXME("should use registry\n");
> - TRACE("polguid %s\n", debugstr_guid((const GUID *)polguid));

You sure like const, don't you?

That cast does nothing, compiler will insert it for you. Better remove it.

Feel free to make the function's argument const, though.