Reduce scope of variables in TobyFitYVector, fixes cppcheck issue.

Refs #8940
mantidproject · Feb 7, 2014 · 2c80cf9 · 2c80cf9
1 parent 087961d
commit 2c80cf9
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/Code/Mantid/Framework/MDAlgorithms/src/Quantification/Resolution/TobyFitYVector.cpp b/Code/Mantid/Framework/MDAlgorithms/src/Quantification/Resolution/TobyFitYVector.cpp
@@ -212,12 +212,13 @@ namespace Mantid
      */
     void TobyFitYVector::calculateSampleContribution()
     {
-      double & sampleBeamDir = m_yvector[TobyFitYVector::ScatterPointBeam];
-      double & samplePerpDir = m_yvector[TobyFitYVector::ScatterPointPerp];
-      double & sampleUpDir = m_yvector[TobyFitYVector::ScatterPointUp];
 
       if(m_sampleVolume)
       {
+        double & sampleBeamDir = m_yvector[TobyFitYVector::ScatterPointBeam];
+        double & samplePerpDir = m_yvector[TobyFitYVector::ScatterPointPerp];
+        double & sampleUpDir = m_yvector[TobyFitYVector::ScatterPointUp];
+
         const Kernel::V3D & boxSize = m_curObs->sampleCuboid();
         sampleBeamDir = boxSize[2]*(nextRandomNumber() - 0.5);
         samplePerpDir = boxSize[0]*(nextRandomNumber() - 0.5);