]> git.saurik.com Git - wxWidgets.git/commitdiff
Work around "possibly uninitialized variable" warning in wxFileName.
authorVadim Zeitlin <vadim@wxwidgets.org>
Wed, 7 Apr 2010 12:43:29 +0000 (12:43 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Wed, 7 Apr 2010 12:43:29 +0000 (12:43 +0000)
Initialize multiplier to its default value in GetHumanReadableSize() to avoid
a "warning: 'multiplier' may be used uninitialized in this function" from g++.

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

src/common/filename.cpp

index 5c457be39cddcce455f7d1770a0b30b3e284f057..34d8845668fcbec68e8a8a7db183eb7f32d0a5a9 100644 (file)
@@ -2656,17 +2656,18 @@ wxString wxFileName::GetHumanReadableSize(const wxULongLong &bs,
 
     // depending on the convention used the multiplier may be either 1000 or
     // 1024 and the binary infix may be empty (for "KB") or "i" (for "KiB")
-    double multiplier;
+    double multiplier = 1024.;
     wxString biInfix;
 
     switch ( conv )
     {
+        case wxSIZE_CONV_TRADITIONAL:
+            // nothing to do, this corresponds to the default values of both
+            // the multiplier and infix string
+            break;
+
         case wxSIZE_CONV_IEC:
             biInfix = "i";
-            // fall through
-
-        case wxSIZE_CONV_TRADITIONAL:
-            multiplier = 1024.;
             break;
 
         case wxSIZE_CONV_SI: