summaryrefslogtreecommitdiff
path: root/vcl
diff options
context:
space:
mode:
authorLuboš Luňák <l.lunak@collabora.com>2018-11-08 13:48:42 +0100
committerLuboš Luňák <l.lunak@collabora.com>2018-11-14 16:05:17 +0100
commit698942ac73b8502eb1957b70dd165023e8d54c82 (patch)
tree946288e160881401a7ded25aa792a2599bf35315 /vcl
parent1fb3d85aa855f57c3a4b32819d0acd9a378bd97c (diff)
avoid valgrind uninitialized jump warning with clang
I'm not sure what exactly the actual problem is, but valgrind reports unitialized reads of mbMulti, although that one is fine and the culprint is in fact mbCenter. Which happens to be bit 16 of the bitfield, and the generated assembly looks fishy, just for that one specific bit. Adding extra padding, extra statements or even just reordering avoids it. So this is lame, but just reorder it and be done with it, this should be otherwise harmless and I've spent enough time on this. Change-Id: I25be55ebe0e9b6dc981793aa7caeae2a40bb27a5 Reviewed-on: https://gerrit.libreoffice.org/63186 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
Diffstat (limited to 'vcl')
-rw-r--r--vcl/inc/listbox.hxx2
1 files changed, 1 insertions, 1 deletions
diff --git a/vcl/inc/listbox.hxx b/vcl/inc/listbox.hxx
index c448c923ed9f..8591d9696764 100644
--- a/vcl/inc/listbox.hxx
+++ b/vcl/inc/listbox.hxx
@@ -215,8 +215,8 @@ private:
bool mbInUserDraw : 1; ///< In UserDraw
bool mbReadOnly : 1; ///< ReadOnly
bool mbMirroring : 1; ///< pb: #106948# explicit mirroring for calc
- bool mbRight : 1; ///< right align Text output
bool mbCenter : 1; ///< center Text output
+ bool mbRight : 1; ///< right align Text output
bool mbEdgeBlending : 1;
Link<ImplListBoxWindow*,void> maScrollHdl;