Differential D3827 Diff 9530 src/common/KReportItemBase.cpp

Changeset View

Standalone View

src/common/KReportItemBase.cpp

Show All 27 Lines
28	{	28		{
29	public:	29		public:
30	Private();	30		Private();
31	~Private();	31		~Private();
32		32
33	KPropertySet *set;	33		KPropertySet *set;
34	KProperty *nameProperty;	34		KProperty *nameProperty;
35	KProperty *sizeProperty;	35		KProperty *sizeProperty;
36	KProperty *positionProperty;	36		KProperty *positionProperty;
			staniekUnsubmitted Done This is not needed. This class of warnings is a false positive. If you mark them as such at coverity.com we don't need any changes and next time it won't complain. Please note that setting nullptr here is misleading. If you want you can change initialization from Private::Private() to an initializer list: : set(new KPropertySet), .... staniek: This is not needed. This class of warnings is a false positive. If you mark them as such at…
37	QString oldName;	37		QString oldName;
38	qreal z;	38		qreal z = 0;
			staniekUnsubmitted Done not needed, it's initialized in the ctor staniek: not needed, it's initialized in the ctor
			piggzAuthorUnsubmitted Done is it better to set to 0 the c++11 way, in Private() or in KReportItemBase() ? piggz: is it better to set to 0 the c++11 way, in Private() or in KReportItemBase() ?
			staniekUnsubmitted Done The c++11 way in Private() if it is only possible. Here it is. Most known exceptions are related to creating QWidgets/QObjects where we have to do that in ctor of the regular class. staniek: The c++11 way in Private() if it is only possible. Here it is. Most known exceptions are…
39	};	39		};
40		40
41	KReportItemBase::Private::Private()	41		KReportItemBase::Private::Private()
42	{	42		{
43	set = new KPropertySet();	43		set = new KPropertySet();
44	nameProperty = new KProperty("name", QString(), tr("Name"), tr("Object Name"));	44		nameProperty = new KProperty("name", QString(), tr("Name"), tr("Object Name"));
45	nameProperty->setAutoSync(0);	45		nameProperty->setAutoSync(0);
46		46
▲ Show 20 Lines • Show All 193 Lines • Show Last 20 Lines