Hello Noel,
Looking at Coverity scan report, it seems nScreen should be initialized.
Indeed, either it's really unused and nScreen should be completely
removed or it's used and its initialization should stay.
If it's first option, I didn't see another patch in gerrit review
concerning vcl part.
Are you sure about loplugin:unusedfields or did I miss something?
Julien
-------- Forwarded Message --------
Subject: New Defects reported by Coverity Scan for LibreOffice
Date: Sat, 17 Jun 2017 21:35:05 -0700
From: scan-admin@coverity.com
To: serval2412@yahoo.fr
Hi,
Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.
3 new defect(s) introduced to LibreOffice found with Coverity Scan.
3 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by
Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 3 of 3 defect(s)
** CID 1412875: Uninitialized members (UNINIT_CTOR)
/vcl/unx/generic/gdi/cairo_xlib_cairo.cxx: 70 in cairo::X11SysData::X11SysData(const SystemGraphicsData
&)()
________________________________________________________________________________________________________
*** CID 1412875: Uninitialized members (UNINIT_CTOR)
/vcl/unx/generic/gdi/cairo_xlib_cairo.cxx: 70 in cairo::X11SysData::X11SysData(const SystemGraphicsData
&)()
64 pRenderFormat(nullptr)
65 {}
66
67 X11SysData::X11SysData( const SystemGraphicsData& pSysDat ) :
68 hDrawable(pSysDat.hDrawable),
69 pRenderFormat(pSysDat.pXRenderFormat)
CID 1412875: Uninitialized members (UNINIT_CTOR)
Non-static class member "nScreen" is not initialized in this constructor nor in any functions
that it calls.
70 {}
71
72 X11SysData::X11SysData( const SystemEnvData& pSysDat ) :
73 pDisplay(pSysDat.pDisplay),
74 hDrawable(pSysDat.aWindow),
75 pVisual(pSysDat.pVisual),
...
Context
- Fwd: New Defects reported by Coverity Scan for LibreOffice · Julien Nabet
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.