[Wine-devel] [Wine-patches] winecfg: Tab to control the loading of key drivers (eterbug #6916). final variant
Alexander Morozov
amorozov на etersoft.ru
Пн Сен 26 11:42:48 MSD 2011
if (doPropertySheet (hInstance, NULL) > 0) {
+ HWND hwnd;
+ if (FixedChanges())
+ if (MessageBoxA(hwnd, "The changes you have made require you to restart all Wine programs.\n\nDo this now?", "Change the
configuration", MB_YESNO | MB_ICONQUESTION) == IDYES)
+ InitiateSystemShutdownExW(NULL, NULL, 0, TRUE, FALSE, SHTDN_REASON_MAJOR_LEGACY_API);
WINE_TRACE("OK\n");
} else {
+ HWND hwnd;
+ if (FixedChanges())
+ if (MessageBoxA(hwnd, "The changes you have made require you to restart all Wine programs.\n\nDo this now?", "Change the
configuration", MB_YESNO | MB_ICONQUESTION) == IDYES)
+ InitiateSystemShutdownExW(NULL, NULL, 0, TRUE, FALSE, SHTDN_REASON_MAJOR_LEGACY_API);
WINE_TRACE("Cancel\n");
}
Зачем это дублирование? Лучше тогда поместить вывод сообщения после if-а.
Длинные строки надо переносить.
Wine везде внутри использует W-функции, так что лучше использовать MessageBoxW.
Вместо hwnd здесь, как я понимаю, надо просто NULL использовать.
Надо сделать для сообщения перевод. А то все надписи на вкладке переведены,
а предупреждение почему-то нет.
Подробная информация о списке рассылки Wine-devel