<div class="gmail_quote">On Tue, Dec 22, 2009 at 9:53 AM, David Doria <span dir="ltr"><<a href="mailto:daviddoria%2Bvtk@gmail.com">daviddoria+vtk@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">Per Ken's suggestion, I am trying to implement a virtual double ComputeSize() function for vtkCell.  There are a lot of cells! Many of which I have no idea how to get their size. Is it acceptable to somehow commit these changes without having all of them implemented? Something like a filler "return 0" and a comment "this function has not yet been implemented"?<div>

<br></div><div>The idea would be that then the framework is setup, so when a developer who is familiar with the particular cell subclass sees the missing implementation, they could easily fill it in.</div><div><br></div>
<div>
Thoughts?<br><div></div></div><div><br></div></div></blockquote><div><br></div><div>I don't think you should commit stuff that's half-(or any other fraction less than 1)-done unless there is somebody who will commit to pounding out the remainder in the very short term (days).</div>
<div><br></div><div>If you can't do all existing subclasses, than it might be a better strategy to develop patches against CVS HEAD and attach them to an issue in the bug tracker until such time as the patch is complete.</div>
<div><br></div><div>Otherwise, we'll end up with a half-done thing in the next release.</div><div><br></div><div>One good idea for ensuring completeness on a task like this is to give the base class virtual a no-op implementation (return -1 even) but to put a "vtkErrorMacro" in it so that when it does get called, the developer who encounters it knows immediately what must be done to eliminate it.</div>
<div><br></div><div>A message like:</div><div>"vtkCell subclass is missing a ComputeSize implementation, please add to " << classname</div><div><br></div><div>would be a great way to do it incrementally via patches in the bug tracker.</div>
<div><br></div><div><br></div><div>HTH,</div><div>David Cole</div><div><br></div></div>