[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