<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 29, 2015 at 2:40 PM, David Cole <span dir="ltr"><<a href="mailto:DLRdave@aol.com" target="_blank">DLRdave@aol.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">(1) what was the problem with double and make_pair -- or was that a<br>
misleading commit message because the problem was always with the use<br>
of NULL ... ?<br></blockquote><div><br></div><div>Nope, it was with the double. See below.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
(2) Will you please rephrase the comment here in a future commit?<br>
<a href="https://gitlab.kitware.com/vtk/vtk/commit/e3d0653de443435533ebfb2db3fa6513e3b5c86b#7d40499821cd2f098d779d9b0da192039787a70a_587_594" target="_blank">https://gitlab.kitware.com/vtk/vtk/commit/e3d0653de443435533ebfb2db3fa6513e3b5c86b#7d40499821cd2f098d779d9b0da192039787a70a_587_594</a></blockquote><div><br></div><div>No can do, it was merged before the complaint was made: <a href="https://github.com/Kitware/VTK/commit/e3d0653de443435533ebfb2db3fa6513e3b5c86b">https://github.com/Kitware/VTK/commit/e3d0653de443435533ebfb2db3fa6513e3b5c86b</a> </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
because it's simply not true. My code snippet proves that MSVC 2013's<br>
make_pair does NOT have a problem with double -- it has a problem with<br>
arguments of poorly defined type. NULL is possibly just "0" under some<br>
circumstances, which has type "int." If you construct a local variable<br>
of the correct type, with value NULL, then make_pair has no problem<br>
deducing the correct type.</blockquote><div><br></div><div>The first issue was with double:</div><div><br></div><div> <a href="https://open.cdash.org/viewBuildError.php?buildid=3784595">https://open.cdash.org/viewBuildError.php?buildid=3784595</a></div><div><br></div><div>error C2664: 'std::pair<double,vtkTextProperty *> std::make_pair<double,vtkTextProperty*>(_Ty1 &&,_Ty2 &&)' : cannot convert argument 1 from 'double' to 'double &&'<br></div><div><br></div><div>Try the change I suggested to your test snippet in the previous email and it should trigger this. There was a discussion on some windows support forum where they said their implementation did strictly follow the standard and they won't change the compiler, which is fair enough. But I don't think anyone would find that behavior intuitive or expected :P</div><div><br></div><div>On Wed, Apr 29, 2015 at 2:37 PM, Will Schroeder <span dir="ltr"><<a href="mailto:will.schroeder@kitware.com" target="_blank">will.schroeder@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">I would say to Dave Cole's point that I have personally seen several problems in the very modest development efforts I have been doing on Windows (e.g., vtkSMPTools related) mostly in the CMake files, etc. Every compiler/platform has something good to say and we need to listen. This ensures that our software remains world class, despite our personal religious beliefs :-) Actually I actually think more of us need to build on Windows/MSVC because stuff is falling through the cracks.</div></blockquote><div><br></div><div>Agreed that all platforms should be well tested and supported. I think the real question with your issue is, why didn't the dashboards catch these problems? Is there some way we could have caught them with a nightly/continuous test?</div><div> </div></div><div>Dave</div></div></div></div>