[Wine-devel] [Wine-patches] shell32: Tab About for eterhack (eterbug #6494). corrected

[Alexander Morozov]

+#include "sys/stat.h"
Системные заголовочные файлы надо включать перед вайновскими и в угловых скобках.

+           WCHAR iexplore[]= {'i','e','x','p','l','o','r','e','.','e','x','e',0};
Тут лучше добавить static const.

+    int hFile;
Название неудачно, т.к. h в начале используется для хэндлов в WinAPI, а это
UNIX-овый дескриптор файла.

+    char BmpFileName[MAX_PATH];
Переменную лучше назвать с маленькой буквы.

+    if ((hFile = open(BmpFileName, O_RDONLY)) == NULL)
open возвращает не указатель, и возврат 0 не означает ошибку.

+    if (SUCCEEDED(CreateStreamOnHGlobal(hMem, FALSE, &pStream)))
+        if (OleLoadPicture(pStream, dwFileSize, TRUE, &IID_IPicture, (void**)&pPicture) != S_OK)
+            return;
+}
Тут нет смысла во втором if и return.
Перед выходом из функции стоит освободить hMem и pStream.