Re: [PUSHED] Some fixes for Clang warnings

2012-04-24 Thread Catalin Iacob
On Mon, Apr 23, 2012 at 2:39 PM, Stephan Bergmann sberg...@redhat.com wrote: Only 0002-WaE-Clang-unused-variable.patch and 0003-WaE-Clang-unused-variable.patch I did not push.  Can you change them as I suggested above (or just let me know if you want me to do that)? I attached updated patches

Re: [PUSHED] Some fixes for Clang warnings

2012-04-23 Thread Stephan Bergmann
On 04/23/2012 01:24 PM, Stephan Bergmann wrote: diff --git a/cppu/source/uno/lbenv.cxx b/cppu/source/uno/lbenv.cxx index 47ca98f..23d4871 100644 --- a/cppu/source/uno/lbenv.cxx +++ b/cppu/source/uno/lbenv.cxx @@ -987,7 +987,7 @@ inline void EnvironmentsData::registerEnvironment( uno_Environment

Re: [PUSHED] Some fixes for Clang warnings

2012-04-23 Thread Lubos Lunak
On Monday 23 of April 2012, Catalin Iacob wrote: The plan is also to report the Clang crash that I encountered, I can report your test too while I'm at it if I can reproduce it (didn't try yet) and then keep going, as time allows, with Clang + werror and see what else turns out. Note that I