<div dir="ltr">+1 to removing it.<div><br></div><div style>Question: is there a way of "watching" certain files in Git? I was wondering if we could set watches for some critical files (such as vtkSetGet) and be notified when they changed?</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 26, 2013 at 8:39 AM, 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="im">On Tue, Mar 26, 2013 at 7:35 AM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>> wrote:<br>


> On Tue, Mar 26, 2013 at 4:38 AM, Will Schroeder<br>
> <<a href="mailto:will.schroeder@kitware.com">will.schroeder@kitware.com</a>> wrote:<br>
>> Hi David-<br>
>><br>
>> On Mon, Mar 25, 2013 at 11:05 PM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>> wrote:<br>
>>><br>
>>> 3) The biggest problem, however, it is that it is a new macro, and if<br>
>>> people plan to add new macros, I think they should check with Berk,<br>
>>> Brad, or Will first.<br>
>><br>
>><br>
>> I disagree on one front, certain changes should be floated past the<br>
>> community (including you) not just a few. However sometimes it's hard to<br>
>> know what these changes are and when to float them, etc. honest mistakes are<br>
>> made. Anyway let's see what we can do about the roach, I need to know more.<br>
><br>
><br>
> I was too hasty to name specific gatekeepers... I agree that these<br>
> decisions are a shared responsibility.  It's worth noting that the new<br>
> macro was added before the switch to Gerrit.<br>
><br>
> My suggestion is that vtkSetNGetMacro be removed, it is only used<br>
> in a couple places and it was not in the VTK 5.10 release.<br>
><br>
> Places where it is used:<br>
> IO/Image/vtkImageReader2.h<br>
> Rendering/Context2D/vtkOpenGLContextDevice2D.h<br>
><br>
</div>I agree that it should be removed, a git grep shows very limited use<br>
of it. I didn't really register it when reviewing code. Review +<br>
mailing list should hopefully catch things like this, but I am<br>
generally much more worried about changes in Common and changes to<br>
module.cmake (big dependency changes).<br>
<br>
Thankfully the new wrapping code looks OK, but the timing had me a<br>
little worried this morning when I looked at the dashboards.<br>
<span class="HOEnZb"><font color="#888888"><br>
Marcus<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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></div>