I don't think it's meaningless. I think it shows that someone has to look at each of these occurrences.<br><br>Please do not underestimate how much work it will take to upgrade an application that will see no immediate benefit other than being able to run the same application with a new version of the code.<br>
<br><div class="gmail_quote">On Thu, May 12, 2011 at 1:50 PM, Berk Geveci <span dir="ltr"><<a href="mailto:berk.geveci@kitware.com">berk.geveci@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
This is a meaningless experiment. I am not removing GetOutput(). A<br>
meaningful experiment would be to look for SetInput() (but not for<br>
SetInputConnection). Then see how many of those instances mean to<br>
establish a pipeline connection as opposed to directly setting a data<br>
object as input. Note that I am not removing SetInput() either, only<br>
changing it such that it does not setup pipeline connection. I would<br>
also discount all instances of:<br>
<br>
foo->SetInput(bar->GetOutput());<br>
<br>
because they can be fixed with a simple script.<br>
<font color="#888888"><br>
-berk<br>
</font><div><div></div><div class="h5"><br>
On Thu, May 12, 2011 at 1:46 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> Berk and Steve,<br>
><br>
> One last Slicer4 experiment. In Slicer4 I grepped for GetOutput() (excluding<br>
> any ITK files). I got 2281 hits.<br>
><br>
> Bill<br>
><br>
><br>
</div></div></blockquote></div><br>