diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-04-17 09:27:25 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-04-17 09:43:17 +0100 |
| commit | 078803a802c86eb772a77074475b5fd3db9b66ce (patch) | |
| tree | 1d9ede6ff09e5f59fde0cd5e72f4b549ff5bf12d | |
| parent | e8c424c3e3ed2717c75c85b4ba36d77767e668b0 (diff) | |
coverity#708655 Uninitialized scalar field
Change-Id: Icbb7d84f127a374f0199c217de529575263a8a4e
| -rw-r--r-- | vcl/unx/generic/dtrans/bmp.cxx | 22 |
1 files changed, 14 insertions, 8 deletions
diff --git a/vcl/unx/generic/dtrans/bmp.cxx b/vcl/unx/generic/dtrans/bmp.cxx index 0e8a2e52426d..0fb3fe6bdee9 100644 --- a/vcl/unx/generic/dtrans/bmp.cxx +++ b/vcl/unx/generic/dtrans/bmp.cxx @@ -346,11 +346,20 @@ void x11::X11_freeBmp( sal_uInt8* pBmp ) * PixmapHolder */ -PixmapHolder::PixmapHolder( Display* pDisplay ) : - m_pDisplay( pDisplay ), - m_aColormap( None ), - m_aPixmap( None ), - m_aBitmap( None ) +PixmapHolder::PixmapHolder( Display* pDisplay ) + : m_pDisplay(pDisplay) + , m_aColormap(None) + , m_aPixmap(None) + , m_aBitmap(None) + , m_nRedShift(0) + , m_nRedShift2(0) + , m_nGreenShift(0) + , m_nGreenShift2(0) + , m_nBlueShift(0) + , m_nBlueShift2(0) + , m_nBlueShift2Mask(0) + , m_nRedShift2Mask(0) + , m_nGreenShift2Mask(0) { /* try to get a 24 bit true color visual, if that fails, * revert to default visual @@ -384,11 +393,8 @@ PixmapHolder::PixmapHolder( Display* pDisplay ) : if( m_aInfo.c_class == TrueColor ) { int nRedSig, nGreenSig, nBlueSig; - m_nRedShift = m_nRedShift2 = 0; getShift( m_aInfo.red_mask, m_nRedShift, nRedSig, m_nRedShift2 ); - m_nGreenShift = m_nGreenShift2 = 0; getShift( m_aInfo.green_mask, m_nGreenShift, nGreenSig, m_nGreenShift2 ); - m_nBlueShift = m_nBlueShift2 = 0; getShift( m_aInfo.blue_mask, m_nBlueShift, nBlueSig, m_nBlueShift2 ); m_nBlueShift2Mask = m_nBlueShift2 ? ~((unsigned long)((1<<m_nBlueShift2)-1)) : ~0L; |