[Wine-devel] Added test for IShellLinkA_GetPatch

Alexander Morozov amorozov на etersoft.ru
Пн Апр 18 16:02:32 MSD 2011


В названии патча надо поправить опечатку в _GetPatch.
Табы лучше не использовать.

+	char buffer[ 256 ];
В wine принято писать так (без пробелов):
char buffer[256];
Вместо 256 лучше использовать MAX_PATH:
char buffer[MAX_PATH];

+	WCHAR addr[] = { 'C', ':', '\\', 'l', 'i', 'n', 'k', '.', 'l', 'n', 'k', 
0 };
Такие штуки принято записывать компактнее, плюс надо добавить static const 
(большие вещи не принято выделять на стеке, то, что не меняется лучше 
объявлять с const):
static const WCHAR abcd[] = {'a','b','\\','c','d',0};

Вместо c:\ лучше использовать временную директорию (см. GetTempPathA).

+	
+	
+	hRes = CoCreateInstance( &CLSID_ShellLink, NULL, CLSCTX_INPROC_SERVER,
ИМХО, две пустые строки здесь излишни, достаточно одной. Табов и пробелов в 
конце строк быть не должно.

+	hRes = IShellLinkA_GetPath( psl, buffer, sizeof(buffer), & pfd, 
SLGP_RAWPATH );
Пробел перед & здесь не нужен.

+	DeleteFile( "c:\\link.lnk" );
Надо явно указывать A и W: DeleteFileA.

Неработающие тесты надо отметить todo_wine.


Подробная информация о списке рассылки Wine-devel