+1 for "<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:12.800000190734863px;background-color:rgb(255,255,255)"> disable copy macro "</span><div><font color="#222222" face="arial, sans-serif">Jc<br>
</font><br><div class="gmail_quote">On Thu, Sep 13, 2012 at 2:29 PM, Aashish Chaudhary <span dir="ltr"><<a href="mailto:aashish.chaudhary@kitware.com" target="_blank">aashish.chaudhary@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 to remove construct and destructor. Also I think like Qt we should provide disable copy macro and use it in vtkClassMacro.<div>
<br></div><div>Thanks, </div><div><div><div class="h5"> <br><br><div class="gmail_quote">On Thu, Sep 13, 2012 at 12:39 PM, Cory Quammen <span dir="ltr"><<a href="mailto:cquammen@cs.unc.edu" target="_blank">cquammen@cs.unc.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 if you remove the constructors/destructor declarations.<br>
<br>
Cory<br>
<div><div><br>
On Thu, Sep 13, 2012 at 11:33 AM, Patchett, John M <<a href="mailto:patchett@lanl.gov" target="_blank">patchett@lanl.gov</a>> wrote:<br>
> Though I don't usually involve myself in VTK development, I think<br>
> that obfuscated understandability is part of the reason that I've seen<br>
> a large number of otherwise bright developers (students) become<br>
> integration impotent when asked to put their ideas/code in<br>
> VTK/ParaView...<br>
> If a goal is to have contributors, this is a bad idea.<br>
><br>
><br>
> --John.<br>
><br>
><br>
> On 9/13/12 9:22 AM, "Marcus D. Hanwell" <<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>> wrote:<br>
><br>
>>On Thu, Sep 13, 2012 at 11:07 AM, Utkarsh Ayachit<br>
>><<a href="mailto:utkarsh.ayachit@kitware.com" target="_blank">utkarsh.ayachit@kitware.com</a>> wrote:<br>
>>>> I am in favor of adding the macro, and disagree with Utkarsh in that<br>
>>>> we already have the type macro etc and it will remove the need for a<br>
>>>> lot of boiler plate code in each class declaration.<br>
>>><br>
>>> Note that vtkTypeMacro() doesn't force you to add anything in the cxx<br>
>>> file. You can pretty much not know what that macro does and you're<br>
>>> okay. Not so with this macro. Also hiding constructor/destructor is<br>
>>> really weird to me. Why would you? I don't think I am thrilled by<br>
>>> having to implement function definitions without seeing their<br>
>>> declarations in the header. None of the existing macros do that, not<br>
>>> even in Qt, as far as I know. Readability of code is far more<br>
>>> important that saving typing a few more lines, IMHO.<br>
>>><br>
>>I can see your point with the contructor/desctructor. Even with them<br>
>>removed I still think this macro would be very useful.<br>
>><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<br>
>><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>
> _______________________________________________<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>
<br>
<br>
</div></div><span><font color="#888888">--<br>
Cory Quammen<br>
Research Associate<br>
Department of Computer Science<br>
The University of North Carolina at Chapel Hill<br>
</font></span><div><div>_______________________________________________<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><br clear="all"><div><br></div></div></div><div class="im">-- <br>| Aashish Chaudhary <br>| R&D Engineer <br>| Kitware Inc. <br>| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a> <br>
</div></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><br clear="all"><div><br></div>-- <br>+1 919 869 8849<br><br>
</div>