<div class="gmail_quote">On Wed, Mar 9, 2011 at 9:47 AM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@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;">
Hi Andrew,<br>
<div class="im"><br>
On Tue, Mar 8, 2011 at 11:13 PM, Andrew Maclean<br>
<<a href="mailto:andrew.amaclean@gmail.com">andrew.amaclean@gmail.com</a>> wrote:<br>
> Hi Marcus,<br>
>    Firstly congratulations to you all on a brilliant first effort.<br>
> One minor change, could I suggest moving:<br>
> [Filters/Sources]<br>
><br>
> Graphics/vtkParametricFunctionSource.cxx:<br>
> Graphics/vtkParametricFunctionSource.h:<br>
><br>
><br>
> To:<br>
> [Core/ComputationalGeometry]<br>
> Logically vtkParametricFunctionSource belongs with the rest of the<br>
> vtkParametric classes.<br>
<br>
</div>This is one case where it would be nice to keep it where it is. The<br>
Core/ComputationalGeometry module should only depend on Core/Core<br>
right now, but if we made this move then it would also depend on<br>
Core/ExecutionModel. Filters/Sources would of course link to<br>
Core/ExecutionModel, and to Core/ComputationalGeometry if necessary.<br>
<br>
That said I have not gotten this far with the build system, but I<br>
think it would be nice to keep the minimal dependency set for<br>
ComputationalGeometry if this is feasible.<br>
<div><div></div><div class="h5"><br>
Marcus<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>
</div></div></blockquote></div><br>Perhaps the other vtkParametric classes belong wherever they need to belong in order to minimize the dependencies then? Perhaps a Parametric module itself somewhere?<br><br>