Few nit picks.
On 10/22/2011 02:53 PM, Lucas Fialho Zawacki wrote:
+static void _write_private_profile_intW(const char *format, WCHAR* section, WCHAR* key, int value, WCHAR* file) +{
- WCHAR* value_str = heap_printfW(format, value);
Here and everywhere else you not checking if memory allocation succeeded.
- WCHAR path[] = {
'%','C','o','m','m','o','n','P','r','o','g','r','a','m','F','i','l','e','s','%','\\','D','i','r','e','c','t','X','\\','D','i','r','e','c','t','I','n','p','u','t','\\','U','s','e','r',' ','M','a','p','s','\0'};
Missing static const qualifiers. Also the above directory does not exists and you not explicitly creating it.
- guid_strA = debugstr_guid(&lpdiaf->guidActionMap);
- guid_size = MultiByteToWideChar(CP_ACP, 0, guid_strA, -1, NULL, 0);
I'm not sure debug function is ok to be used in the regular code. Also, if need it as unicode why not print it as unicode from the beginning?
- IDirectInputDevice_GetDeviceInfo(&This->IDirectInputDevice8W_iface, &didevi);
You should use IDirectInputDevice8W_from_impl() function instead of directly accessing internals.
memcpy(username, find_data.cFileName, (end - find_data.cFileName + 1) * sizeof(WCHAR));
No checking for big enough buffer. Assuming that file names are in specified format.
Also I'm not exactly sure why do you need a mapping user name anyway? Wine does not support multiple users. The user currently running Wine would be the one created all the files. All this seems to be redundant.
Vitaliy.
-
Vitaliy Margolen