<div dir="ltr">Hi Niels,<div><br></div><div>This does sound like a good patch. Would you mind contributing it via our gitlab contribution process? Please see [1].</div><div><br></div><div>[1] <a href="https://gitlab.kitware.com/vtk/vtk/blob/master/CONTRIBUTING.md">https://gitlab.kitware.com/vtk/vtk/blob/master/CONTRIBUTING.md</a><br></div><div><br></div><div>Thank you,</div><div>Cory</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 23, 2015 at 9:46 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">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', in various CMake files of the VTK source tree, according to the patch/diff-file I attached last Sunday.<div><div class="h5"><br>
<br>
On Sunday, September 20, I wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
When building VTK within Microsoft Visual Studio, I always get many<br>
(20+) 'D9025' warnings, saying:<br>
<br>
   "cl : Command line warning D9025: overriding '/W1' with '/w'"<br>
<br>
These warnings appear useless and distracting to me. Can someone<br>
please fix this issue?<br>
<br>
The issue can be fixed by using '/W0', instead of '-w' or '/w', to<br>
disable MSVC warnings. It should be fixed in the following six files:<br>
<br>
   CMake\vtkModuleMacros.cmake<br>
   CMake\vtkThirdParty.cmake<br>
   ThirdParty\xdmf2\vtkxdmf2\CMake\vtkThirdParty.cmake<br>
   ThirdParty\exodusII\vtkexodusII\CMakeLists.txt<br>
   ThirdParty\hdf5\vtkhdf5\CMakeLists.txt<br>
   ThirdParty\netcdf\vtknetcdf\CMakeLists.txt<br>
<br>
I encountered these warnings on any version of VTK, CMake, and Visual<br>
Studio that I tried recently, including the latest releases, VTK<br>
6.3.0, CMake 3.3.2, and Visual Studio 2015 (but also Visual Studio 2010).<br>
<br>
ITK used to have the same issue. My fix for the ITK issue was included<br>
with ITK 4.8.0. It was discussed at the ITK users mailing list:<br>
<a href="http://public.kitware.com/pipermail/community/2015-February/008349.html" rel="noreferrer" target="_blank">http://public.kitware.com/pipermail/community/2015-February/008349.html</a> My<br>
ITK patch is at: <a href="http://review.source.kitware.com/#/c/19311/" rel="noreferrer" target="_blank">http://review.source.kitware.com/#/c/19311/</a><br>
<br>
A patch (unified diff) for VTK is hereby attached. Can someone please<br>
commit + push it to Kitware's VTK GIT, for me?<br>
</blockquote>
<br>
<br>
Kind regards, Niels<br>
-- <br>
Niels Dekker<br>
Scientific programmer<br>
LKEB, Division of Image Processing<br>
Leiden University Medical Center<br>
<br>
<br></div></div>
_______________________________________________<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>
</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>