FRP wrappers. "Update" menu item.

src/engine/main.cpp

Index: src/engine/main.cpp
===================================================================
--- a/src/engine/main.cpp
+++ b/src/engine/main.cpp
@@ -40,7 +40,7 @@
     }
   }
   
-  bool updateAvailable;
+  bool updateAvailable = false;
   void UpdateCallback(const std::string res)
   {
     if (updateAvailable)
@@ -50,16 +50,15 @@
     {
       std::wstring upToDateText = dictionary->Lookup("updater", "update-already-up-to-date-text");
       std::wstring upToDateTitle = dictionary->Lookup("updater", "update-already-up-to-date-title");
-      MessageBox(NULL, upToDateText.c_str(), upToDateTitle.c_str(), MB_OK);
+      MessageBox(NULL, upToDateText.c_str(), upToDateTitle.c_str(), MB_ICONINFORMATION);

[Wladimir Palant, 2013/08/13 09:19:51]

Not sure why but this change isn't present in the current code. But I agree with
Felix, it should be MB_OK | MB_ICONINFORMATION.

     }
     else
     {
       std::wstring errorText = dictionary->Lookup("updater", "update-error-text");
       std::wstring errorTitle = dictionary->Lookup("updater", "update-error-title");
       ReplaceString(errorText, L"?1?", ToUtf16String(res));
-      MessageBox(NULL, errorText.c_str(), errorTitle.c_str(), MB_OK);
+      MessageBox(NULL, errorText.c_str(), errorTitle.c_str(), MB_ICONEXCLAMATION);

[Wladimir Palant, 2013/08/13 09:19:51]

Not sure why but this change isn't present in the current code. But I agree with
Felix, it should be MB_OK | MB_ICONEXCLAMATION.

     }
-    return;

[Wladimir Palant, 2013/08/13 09:19:51]

This change isn't present in the current code.

   }