Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

MouseSap: silence Valgrind

(1/2) > >>

tomjnx:
This trivial patch reduces the number of Valgrind "conditional jump depends on uninitialized value" warnings:

diff --git a/src/plugins/contrib/MouseSap/MouseSap.cpp b/src/plugins/contrib/MouseSap/MouseSap.cpp
index e611f18..dbf3a74 100644
--- a/src/plugins/contrib/MouseSap/MouseSap.cpp
+++ b/src/plugins/contrib/MouseSap/MouseSap.cpp
@@ -64,6 +64,7 @@ MouseSap* MouseSap::pMouseSap;
 
 // ----------------------------------------------------------------------------
 MouseSap::MouseSap()
+     : m_pMyLog(0), m_bEditorsAttached(false), m_bMouseSapEnabled(false), m_pMMSapEvents(0)
 // ----------------------------------------------------------------------------
 {
        //ctor

oBFusCATed:
As far as I can see the patch is broken, also please use nullptr instead of 0 for pointers.

tomjnx:

--- Quote from: oBFusCATed on January 05, 2013, 06:42:30 pm ---As far as I can see the patch is broken, also please use nullptr instead of 0 for pointers.

--- End quote ---

Ok, fixed. Also, this patch is probably too trivial to apply it using patch 8-)

diff --git a/src/plugins/contrib/MouseSap/MouseSap.cpp b/src/plugins/contrib/MouseSap/MouseSap.cpp
index e611f18..77bde4d 100644
--- a/src/plugins/contrib/MouseSap/MouseSap.cpp
+++ b/src/plugins/contrib/MouseSap/MouseSap.cpp
@@ -64,6 +64,7 @@ MouseSap* MouseSap::pMouseSap;
 
 // ----------------------------------------------------------------------------
 MouseSap::MouseSap()
+     : m_pMyLog(nullptr), m_bEditorsAttached(false), m_bMouseSapEnabled(false), m_pMMSapEvents(nullptr)
 // ----------------------------------------------------------------------------
 {
        //ctor

oBFusCATed:
In SVN...

BTW: How long does it take to ryn cb under valgrind?

tomjnx:

--- Quote from: oBFusCATed on January 05, 2013, 07:05:54 pm ---BTW: How long does it take to ryn cb under valgrind?

--- End quote ---

Thanks a lot!

It takes approximately 90 seconds from invocation to display the main window on an:
Intel(R) Core(TM)2 Duo CPU     E8400  @ 3.00GHz
(with core plugins installed only)

Afterwards, editing is noticeably slower, but bearable for bug hunting...

Navigation

[0] Message Index

[#] Next page