[PATCH] clang, -Wuninitialized

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Sun Jun 2 13:19:14 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/4137

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/37/4137/1

clang, -Wuninitialized

Change-Id: Ib7c585c227d999870d5277cbee0f5bfd562185c3
---
M bridges/source/cpp_uno/gcc3_linux_intel/callvirtualmethod.cxx
M bridges/source/cpp_uno/gcc3_solaris_intel/uno2cpp.cxx
M bridges/source/cpp_uno/mingw_intel/callvirtualmethod.cxx
3 files changed, 3 insertions(+), 3 deletions(-)



diff --git a/bridges/source/cpp_uno/gcc3_linux_intel/callvirtualmethod.cxx b/bridges/source/cpp_uno/gcc3_linux_intel/callvirtualmethod.cxx
index af060de..87384b8 100644
--- a/bridges/source/cpp_uno/gcc3_linux_intel/callvirtualmethod.cxx
+++ b/bridges/source/cpp_uno/gcc3_linux_intel/callvirtualmethod.cxx
@@ -63,7 +63,7 @@
 #endif
 
     volatile long edx = 0, eax = 0; // for register returns
-    void * stackptr;
+    void * stackptr = 0;
     asm volatile (
         "mov   %%esp, %6\n\t"
         // preserve potential 128bit stack alignment
diff --git a/bridges/source/cpp_uno/gcc3_solaris_intel/uno2cpp.cxx b/bridges/source/cpp_uno/gcc3_solaris_intel/uno2cpp.cxx
index 1be1215..532acb4 100644
--- a/bridges/source/cpp_uno/gcc3_solaris_intel/uno2cpp.cxx
+++ b/bridges/source/cpp_uno/gcc3_solaris_intel/uno2cpp.cxx
@@ -58,7 +58,7 @@
     if (! pAdjustedThisPtr) CPPU_CURRENT_NAMESPACE::dummy_can_throw_anything("xxx"); // address something
 
     volatile long edx = 0, eax = 0; // for register returns
-    void * stackptr;
+    void * stackptr = 0;
     asm volatile (
         "mov   %%esp, %6\n\t"
         // copy values
diff --git a/bridges/source/cpp_uno/mingw_intel/callvirtualmethod.cxx b/bridges/source/cpp_uno/mingw_intel/callvirtualmethod.cxx
index c1bb9b6..eece774 100644
--- a/bridges/source/cpp_uno/mingw_intel/callvirtualmethod.cxx
+++ b/bridges/source/cpp_uno/mingw_intel/callvirtualmethod.cxx
@@ -57,7 +57,7 @@
     if (! pAdjustedThisPtr) CPPU_CURRENT_NAMESPACE::dummy_can_throw_anything("xxx"); // address something
 
     volatile long edx = 0, eax = 0; // for register returns
-    void * stackptr;
+    void * stackptr = 0;
     asm volatile (
         "mov   %%esp, %6\n\t"
         // copy values

-- 
To view, visit https://gerrit.libreoffice.org/4137
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib7c585c227d999870d5277cbee0f5bfd562185c3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet <serval2412 at yahoo.fr>



More information about the LibreOffice mailing list