No changes, just refactor the code in MSW wxGetOsDescription() slightly.

Construct the description string from several pieces: the OS name, its build
number and any extra information about it, instead of duplicating the code
appending the build number to the name in several places.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@67753 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/msw/utils.cpp b/src/msw/utils.cpp
index d371459b0909fe9b566208dc6c90a159b8f7138a..f208feb48fa6c50811d15f11cb9782614d940414 100644 (file)
--- a/src/msw/utils.cpp
+++ b/src/msw/utils.cpp
@@ -1274,8 +1274,7 @@ wxString wxGetOsDescription()
                         switch ( info.dwMinorVersion )
                         {
                             case 0:
                         switch ( info.dwMinorVersion )
                         {
                             case 0:

-                                str.Printf(_("Windows 2000 (build %lu"),
-                                           info.dwBuildNumber);
+                                str = _("Windows 2000");

                                 break;
 
                             case 2:
                                 break;
 
                             case 2:


@@ -1284,15 +1283,13 @@ wxString wxGetOsDescription()
                                 // type to resolve this ambiguity
                                 if ( wxIsWindowsServer() == 1 )
                                 {
                                 // type to resolve this ambiguity
                                 if ( wxIsWindowsServer() == 1 )
                                 {

-                                    str.Printf(_("Windows Server 2003 (build %lu"),
-                                               info.dwBuildNumber);
+                                    str = _("Windows Server 2003");

                                     break;
                                 }
                                 //else: must be XP, fall through
 
                             case 1:
                                     break;
                                 }
                                 //else: must be XP, fall through
 
                             case 1:

-                                str.Printf(_("Windows XP (build %lu"),
-                                           info.dwBuildNumber);
+                                str = _("Windows XP");

                                 break;
                         }
                         break;
                                 break;
                         }
                         break;


@@ -1301,29 +1298,15 @@ wxString wxGetOsDescription()
                         switch ( info.dwMinorVersion )
                         {
                             case 0:
                         switch ( info.dwMinorVersion )
                         {
                             case 0:

-                                if ( wxIsWindowsServer() == 1 )
-                                {
-                                    str.Printf(_("Windows Server 2008 (build %lu"),
-                                               info.dwBuildNumber);
-                                }
-                                else
-                                {
-                                     str.Printf(_("Windows Vista (build %lu"),
-                                               info.dwBuildNumber);
-                                }
+                                str = wxIsWindowsServer() == 1
+                                        ? _("Windows Server 2008")
+                                        : _("Windows Vista");

                                 break;
 
                             case 1:
                                 break;
 
                             case 1:

-                                if ( wxIsWindowsServer() == 1 )
-                                {
-                                    str.Printf(_("Windows Server 2008 R2 (build %lu"),
-                                               info.dwBuildNumber);
-                                }
-                                else
-                                {
-                                     str.Printf(_("Windows 7 (build %lu"),
-                                               info.dwBuildNumber);
-                                }
+                                str = wxIsWindowsServer() == 1
+                                        ? _("Windows Server 2008 R2")
+                                        : _("Windows 7");

                                 break;
                         }
                         break;
                                 break;
                         }
                         break;


@@ -1331,12 +1314,13 @@ wxString wxGetOsDescription()
 
                 if ( str.empty() )
                 {
 
                 if ( str.empty() )
                 {

-                    str.Printf(_("Windows NT %lu.%lu (build %lu"),
-                           info.dwMajorVersion,
-                           info.dwMinorVersion,
-                           info.dwBuildNumber);
+                    str.Printf(_("Windows NT %lu.%lu"),
+                               info.dwMajorVersion,
+                               info.dwMinorVersion);

                 }
 
                 }
 

+                str << wxT(" (")
+                    << wxString::Format(_("build %lu"), info.dwBuildNumber);

                 if ( !wxIsEmpty(info.szCSDVersion) )
                 {
                     str << wxT(", ") << info.szCSDVersion;
                 if ( !wxIsEmpty(info.szCSDVersion) )
                 {
                     str << wxT(", ") << info.szCSDVersion;