Yes, that's what I meant by the new factory pattern.<br><br>For now, I'll push a patch that removes VTK_USE_METAIO. The factory stuff is a bit above my pay grade right now.<br><br>Bill<br><br><br><div class="gmail_quote">
On Sun, Nov 4, 2012 at 1:12 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.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="HOEnZb"><div class="h5">On Sun, Nov 4, 2012 at 7:54 AM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> Folks,<br>
><br>
> The vtkImageReader2Factory fails to register vtkMetaImageReader if<br>
> VTK_USE_METAIO is not defined. In VTK6, VTK_USE_METAIO is only referenced in<br>
> <a href="http://vtkToolkits.h.in" target="_blank">vtkToolkits.h.in</a> and vtkImageReader2Factory.cxx.<br>
><br>
> Should VTK_USE_METAIO be removed?<br>
><br>
> My guess is that vtkImageReader2Factory needs to be rewritten to use the new<br>
> factory pattern.<br>
><br>
</div></div>Hi Bill,<br>
<br>
It looks like I missed that one, but yes any remaining VTK_USE_ should<br>
be evaluated as a module should not change its API or implementation<br>
based on compile flags. There are several exceptions to this in<br>
rendering where things were too tightly coupled to separate without a<br>
big rewrite (and little gain).<br>
<br>
The approach we took with the SQL modules should work well here<br>
shouldn't it? I am guessing that is what you mean by new factory<br>
pattern.<br>
<span class="HOEnZb"><font color="#888888"><br>
Marcus<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>