PixelGeoModel: Fix cppcheck warnings.

• Disable copy ctors/assignment for some classes that manage memory.
• Pass class instances via const reference, not by value.
• Prefer using an initialization list to initializing members in a ctor body.
• Prefer preincrement to postincrement for iterators.

This merge request affects 1 package:

• InnerDetector/InDetDetDescr/PixelGeoModel

Adding @goetz ,@lshan ,@battagl ,@sroe ,@stsuno as watchers

added InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash 58318761)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21856]

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 1f0b49d1

added sweep:ignore label