<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 29, 2015 at 3:54 PM, David Lonie <span dir="ltr"><<a href="mailto:david.lonie@kitware.com" target="_blank">david.lonie@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 dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Wed, Apr 29, 2015 at 3:44 PM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.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"><span>On Wed, 29 Apr 2015 13:50:53 -0400, David Cole via vtk-developers said:<br>
<br>
>claims there was some sort of problem with make_pair and double. What<br>
>exactly was the problem? With what exact version of MSVC? (I don't buy<br>
>that there's a general problem with make_pair and double...)<br>
<br>
</span>I'll steer clear of the larger point... :)  but will just say: when adding workarounds for problematic compilers / OSes, it would be very appreciated to be clear about which versions.  As someone who has gone through VTK to expunge obsolete workarounds, it's sometimes been hard to know what's obsolete and what's not.  A lot of workaround notes say things like "on Apple this is broken", but which OS version is not stated...<br>
<br>
Try to think of the future in commit messages and code comments.  :)</blockquote><div><br></div></span><div>+1. In this case, the workaround is still valid code, no alternate paths, and just a slight change in syntax, so we won't have to worry about removing it in the future :D I did leave a note in-line warning future maintainers that the line is troublesome for MSVC and should be treated with caution.</div><div><br></div><div>I am somewhat curious what version of the compiler this was too, I can't really tell from the CDash page:</div><div><br></div><div><a href="https://open.cdash.org/viewBuildError.php?buildid=3784595" target="_blank">https://open.cdash.org/viewBuildError.php?buildid=3784595</a></div></div></div></div></blockquote><div><br></div><div>It was MSVC 13, for anyone who's curious.</div><div><br></div><div>Dave </div></div></div></div>