BUG: Moving PropageRequestedRegion() at the correct place in ImageList, and...

BUG: Moving PropageRequestedRegion() at the correct place in ImageList, and remove the unnecessary call in PipelineMemoryPrintCalculator (part of fir for #1516)

Modules/Core/ObjectList/include/otbImageList.txx

@@ -33,6 +33,19 @@ ImageList<TImage>
 ::UpdateOutputData()
 {
   Superclass::UpdateOutputData();
+  for (ConstIterator it = this->Begin(); it != this->End(); ++it)
+    {
+        it.Get()->GetSource()->UpdateOutputData(it.Get());
+    }
+}
+
+template <class TImage>
+void
+ImageList<TImage>
+::PropagateRequestedRegion() throw (itk::InvalidRequestedRegionError)
+  {
+  Superclass::PropagateRequestedRegion();
+
   for (ConstIterator it = this->Begin(); it != this->End(); ++it)
     {
     if (it.Get()->GetUpdateMTime() < it.Get()->GetPipelineMTime()
@@ -42,7 +55,7 @@ ImageList<TImage>
       if (it.Get()->GetSource())
         {
         it.Get()->GetSource()->PropagateRequestedRegion(it.Get());
-
+        
         // Check that the requested region lies within the largest possible region
         if (!it.Get()->VerifyRequestedRegion())
           {
@@ -51,22 +64,12 @@ ImageList<TImage>
           e.SetLocation(ITK_LOCATION);
           e.SetDataObject(it.Get());
           e.SetDescription("Requested region is (at least partially) outside the largest possible region.");
-
+          
           throw e;
           }
-
-        it.Get()->GetSource()->UpdateOutputData(it.Get());
         }
       }
     }
-}
-
-template <class TImage>
-void
-ImageList<TImage>
-::PropagateRequestedRegion() throw (itk::InvalidRequestedRegionError)
-  {
-  Superclass::PropagateRequestedRegion();
   }
 
 template <class TImage>

Modules/Core/Streaming/src/otbPipelineMemoryPrintCalculator.cxx

@@ -187,11 +187,8 @@ PipelineMemoryPrintCalculator
        it != imageList->End(); ++it)                                    \
        {                                                             \
        if(it.Get()->GetSource())                                        \
-         {                                                              \
-         it.Get()->PropagateRequestedRegion();                                   \
          print += this->EvaluateProcessObjectPrintRecursive(it.Get()->GetSource());\
-         }                                                              \
-else                                                                    \
+       else                                                             \
          print += this->EvaluateDataObjectPrint(it.Get());              \
        }                                                           \
     return print;                                                  \