Fix for a Coverity warning (Ruben)
[u/mrichter/AliRoot.git] / ITS / AliITSAlignMille2ConstrArray.cxx
index b4ca683..84c0b34 100644 (file)
@@ -34,7 +34,7 @@ AliITSAlignMille2ConstrArray::AliITSAlignMille2ConstrArray(const AliITSAlignMill
 {/* DUMMY */} 
 
 //________________________________________________________________________________________________________
-void AliITSAlignMille2ConstrArray::AddModule(AliITSAlignMille2Module* mod)
+void AliITSAlignMille2ConstrArray::AddModule(AliITSAlignMille2Module* mod, Bool_t needGeom)
 {
   int nmd = GetNModules();
   // check if its already not there
@@ -42,20 +42,22 @@ void AliITSAlignMille2ConstrArray::AddModule(AliITSAlignMille2Module* mod)
   fModuleIDs.Set(nmd+1);
   fModulePatt.Set(nmd+1);
   fModuleIDs[nmd] = mod->GetUniqueID();
-  double jacobian[AliITSAlignMille2Module::kMaxParGeom][AliITSAlignMille2Module::kMaxParGeom];
-  if (mod->GeomParamsGlobal()) mod->CalcDerivLocGlo(&jacobian[0][0]);
-  //
-  Short_t patt =  GetPattern();
-  if (mod->GeomParamsGlobal()) {
-    // the constraint is defined in the module's local frame. If the alignment of geom params is
-    // done in the global frame, we need to set the real parameter involved
-    patt &= 0xffff<< AliITSAlignMille2Module::kMaxParGeom;     // reset the geometry parameters
-    for (int i=0;i<AliITSAlignMille2Module::kMaxParGeom;i++) {
-      if (!IncludesParam(i)) continue;
-      for (int j=0;j<AliITSAlignMille2Module::kMaxParGeom;j++) if (jacobian[i][j]!=0) patt |= (0x1<<j);
+  if (needGeom) { // this is geometrical constraint
+    double jacobian[AliITSAlignMille2Module::kMaxParGeom][AliITSAlignMille2Module::kMaxParGeom];
+    if (mod->GeomParamsGlobal()) mod->CalcDerivLocGlo(&jacobian[0][0]);
+    //
+    Short_t patt =  GetPattern();
+    if (mod->GeomParamsGlobal()) {
+      // the constraint is defined in the module's local frame. If the alignment of geom params is
+      // done in the global frame, we need to set the real parameter involved
+      patt &= (Short_t) 0xffff & (0xffff<< AliITSAlignMille2Module::kMaxParGeom);     // reset the geometry parameters
+      for (int i=0;i<AliITSAlignMille2Module::kMaxParGeom;i++) {
+       if (!IncludesParam(i)) continue;
+       for (int j=0;j<AliITSAlignMille2Module::kMaxParGeom;j++) if (jacobian[i][j]!=0) patt |= (0x1<<j);
+      }
     }
+    fModulePatt[nmd] = patt;
   }
-  fModulePatt[nmd] = patt;
 }
 
 //________________________________________________________________________________________________________
@@ -80,7 +82,7 @@ Bool_t AliITSAlignMille2ConstrArray::IncludesModPar(Int_t id,Int_t par) const
 }
 
 //________________________________________________________________________________________________________
-Bool_t AliITSAlignMille2ConstrArray::IncludesModPar(AliITSAlignMille2Module* mod, Int_t par) const
+Bool_t AliITSAlignMille2ConstrArray::IncludesModPar(const AliITSAlignMille2Module* mod, Int_t par) const
 {
   // is this module/parameter mentioned in the list?
   return IncludesModPar(mod->GetUniqueID(), par);