<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">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 class="">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><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">https://open.cdash.org/viewBuildError.php?buildid=3784595</a></div><div><br></div><div>IIRC, we used to have this prominently displayed in the build name. Looks like that got dropped somewhere along the way. I'll see if we can get that info back in.</div><div><br></div><div>Dave</div><div> </div></div></div></div>