<div dir="ltr">Niels,<div><br></div><div>I saw that you contributed to ITK via gerrit. Contributing to VTK is similar, but it uses gitlab instead of gerrit. I wouldn't bother with a bug report for this issue - just explaining why the warnings appear in the commit message is sufficient.</div><div><br></div><div>Thanks for offering to contribute! I will watch for your patch and shepherd it through the review process.</div><div><br></div><div>Best,</div><div>Cory</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 23, 2015 at 10:16 AM, Niels Dekker - address until 2018 <span dir="ltr"><<a href="mailto:niels_dekker_address_until_2018@xs4all.nl" target="_blank">niels_dekker_address_until_2018@xs4all.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Cory,<br>
<br>
Thank you! When I have time (hopefully before next Sunday), I'll try to contribute the patch via your gitlab contribution process. Is the process similar to ITK? And do you think I should create a "ticket" or "bug report" for this little issue?<br>
<br>
If I can't get it to work (or if it takes me too much time), I hope someone else can do the patch for me, next week or so...<br>
<br>
  Kind regards, Niels<br>
<br>
PS Also thanks to Audrius for the support  :-)<span class=""><br>
<br>
Cory Quammen wrote on September 23, 2015, 15:54:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
Hi Niels,<br>
<br>
This does sound like a good patch. Would you mind contributing it via<br>
our gitlab contribution process? Please see [1].<br>
<br>
[1] <a href="https://gitlab.kitware.com/vtk/vtk/blob/master/CONTRIBUTING.md" rel="noreferrer" target="_blank">https://gitlab.kitware.com/vtk/vtk/blob/master/CONTRIBUTING.md</a><br>
<br>
Thank you,<br>
Cory<br>
<br></span><span class="">
On Wed, Sep 23, 2015 at 9:46 AM, Niels Dekker wrote:<br>
<br>
    Does anyone here think it would be nice to fix these MSVC warnings?<br>
<br>
    Basically it's just a matter of replacing '/w' and '-w' by '/W0',<br>
    in various CMake files of the VTK source tree, according to the<br>
    patch/diff-file I attached last Sunday.<br>
</span></blockquote>
<br><div class="HOEnZb"><div class="h5">
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the VTK FAQ at: <a href="http://www.vtk.org/Wiki/VTK_FAQ" rel="noreferrer" target="_blank">http://www.vtk.org/Wiki/VTK_FAQ</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtkusers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=vtkusers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtkusers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/vtkusers</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Cory Quammen<br>R&D Engineer<br>Kitware, Inc.</div>
</div>