Hi Berk, <br><br>I'm okay with your naming suggestion. I'll use vtkAbstractPolyDataReader . I'll refrain from using 2 in the Collection and Factory. Does the word Abstract belong in those two? My feeling is not. Too bad about not being able to rename ImageReader2-related classes to Abstract-something. <br>
<br>In keeping with Brad's response, I'll maintain the derivation from vtkPolyDataAlgorithm. Moreover, I want to proceed in as similar to ImageReader2 as possible, without reinventing the hierarchy. <br><br>I'd like to commit my code this afternoon. I have a few test files as well that use stl and vtk data in VTKData. <br>
<br>Cheers, <br><br>Michel<br><br><br><div class="gmail_quote">On Fri, Jan 29, 2010 at 2:17 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="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">> In response to Berk, my justification for calling this reader<br>
> vtkPolyDataReader2 was based on a notion of backward compatibility,<br>
> given that this reader is exactly analogous to vtkImageReader2.<br>
<br>
</div>Backwards compatibility is not the right term. You mean consistency.<br>
Being consistent with a bad precedent is not necessarily the right thing.<br>
<div class="im"><br>
> I'm open to calling it vtkFlexiblePolyDataReader or something in that<br>
> spirit, but some thought should be given to renaming vtkImageReader2<br>
<br>
</div>vtkFlexiblePolyDataReader is slightly better but still quite bad. We have<br>
been using "Abstract" or "Base" to represent abstract superclasses. So<br>
it needs to be vtkAbstractPolyDataReader or vtkPolyDataReaderBase.<br>
<br>
Renaming vtkImageReader2 has the problem of breaking backwards<br>
compatibility. I would love to see it renamed but I am not sure if it is<br>
worth breaking all of the external code that uses it.<br>
<div class="im"><br>
> in a comparable manner, otherwise the naming convention becomes even<br>
> more confusing. Moreover, it's been suggested that I also implement<br>
> such a reader for vtkUnstructuredGrid, so one option would be to call<br>
> all of these vtkXXXReader2 for now, and vtkFlexibleReader at the next<br>
> release.<br>
<br>
</div>I did not suggest implementing a base for unstructured grid readers. I<br>
suggested implementing a base of all readers, something like<br>
vtkAbstractDataReader.<br>
ALL VTK readers would be a descendant of this class.<br>
<font color="#888888"><br>
-berk<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Michel Audette, Ph.D. <br>R & D Engineer, <br>Kitware Inc.,<br>Chapel Hill, N.C. <br><br>