[Wine-devel] [Wine-patches] eterbag6846: ole32/test: Incorrect storage creation

[amorozov на etersoft.ru]

Правильно писать не eterbag, а eterbug.
Поддиректория ole32 называется tests, а не test, так что надо писать ole32/tests.

+    ok(r == S_OK, "CreateILockBytesOnHGlobal failed, hr=%08x\n", r);
+    GlobalUnlock(hMem);
GlobalUnlock здесь не нужен, т.к. мы не вызывали GlobalLock.

+    ok(!memcmp( pFileBuffer, StandardMagic, 8), "Incorrect storage creation \n");
Пробел перед \n не нужен.
Надо добавить ещё какую-нибудь проверку на обнуление поля, которое до этого было заполнено 0xff.

+    GetHGlobalFromILockBytes( ilb, &hMem );
+    pFileBuffer = GlobalLock(hMem);
Лучше соблюдать какой-то один стиль кодирования, т.е. либо ставить пробелы у скобок, либо нет.
Вообще, то, что сейчас в dlls/ole32/tests/storage32.c - плохой пример соблюдения стиля
кодирования, но нам лучше всё же как-то его соблюдать, чтобы уменьшить вероятность игнорирования
нашего патча апстримом.