<br><br><div class="gmail_quote">On Tue, Jun 12, 2012 at 7:35 AM, David Doria <span dir="ltr"><<a href="mailto:daviddoria@gmail.com" target="_blank">daviddoria@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote">On Tue, Jun 12, 2012 at 7:23 AM, Dženan Zukić <span dir="ltr"><<a href="mailto:dzenanz@gmail.com" target="_blank">dzenanz@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<font><font face="verdana,sans-serif">Probably to allow the flexibility of activating it at time different from setting the file name?</font></font></blockquote><div><br></div>Even if that is the case, why not just fileOutputWindow->Activate() (with no arguments) or something like that? It seems confusing to have to pass an object to itself.</div>
</blockquote><div><br></div><div>You're not passing it to itself, you're calling a *static* method to set a static data member with an instance pointer.</div><div><br></div><div>It would be better phrased as:</div>
<div><br></div><div>  vtkOutputWindow::SetInstance(fileOutputWindow);</div><div><br></div><div>rather than:</div><div><br></div><div>  fileOutputWindow->SetInstance(fileOutputWindow);</div><div><br></div><div>because the "class::static_method" phrasing reflects what's actually happening.</div>
<div><br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><span class="HOEnZb"><font color="#888888"><br clear="all">

<br>David </font></span></div>
<br>_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br>