<div dir="ltr"><span class="">FYI: A workaround of Niels was submitted to gerrit (by Marius Staring). Please have a look at the patch.<br></span><div><span class=""><br>I8177d3b7: BUG: workaround for 64bit VS2013 Release segfault<br><a href="http://review.source.kitware.com/#/c/18825/">http://review.source.kitware.com/#/c/18825/</a><br><br></span></div><div><span class="">Thanks,<br>Floris<br></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 9, 2015 at 4:34 PM, Matt McCormick <span dir="ltr"><<a href="mailto:matt.mccormick@kitware.com" target="_blank">matt.mccormick@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Niels,<br>
<br>
Great news!<br>
<br>
This same strategy was used to to fix some of the other failing tests<br>
[1]. While it should not strictly be necessary, it does hurt to<br>
improve our const-correctness and memory usage anyway.  Please<br>
consider submitting a patch for these changes [2].<br>
<br>
Thanks,<br>
Matt<br>
<br>
<br>
[1]  <a href="http://itk.org/gitweb?p=ITK.git;a=blobdiff;f=Modules/Filtering/ImageGrid/include/itkShrinkImageFilter.hxx;h=fca11a3eea3f885411192fd942c674cbba30c604;hp=86c3985601ff24ee36f6a0972be11573dfcc277c;hb=b92b952b98bee6dfc3837f4ed5d600e637cdcda9;hpb=7170ea77a918d268dd1971244ceb41b6d3518e21" target="_blank">http://itk.org/gitweb?p=ITK.git;a=blobdiff;f=Modules/Filtering/ImageGrid/include/itkShrinkImageFilter.hxx;h=fca11a3eea3f885411192fd942c674cbba30c604;hp=86c3985601ff24ee36f6a0972be11573dfcc277c;hb=b92b952b98bee6dfc3837f4ed5d600e637cdcda9;hpb=7170ea77a918d268dd1971244ceb41b6d3518e21</a><br>
<br>
[2] <a href="https://insightsoftwareconsortium.github.io/ITKBarCamp-doc/CommunitySoftwareProcess/SubmitAPatchToGerrit/index.html" target="_blank">https://insightsoftwareconsortium.github.io/ITKBarCamp-doc/CommunitySoftwareProcess/SubmitAPatchToGerrit/index.html</a><br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Jan 9, 2015 at 10:28 AM, Niels Dekker <<a href="mailto:niels-xtk@xs4all.nl">niels-xtk@xs4all.nl</a>> wrote:<br>
> Floris and I did some testing on how to work around this Visual C++<br>
> compiler bug. It looks like the crashes (segfaults) in both<br>
> FiniteDifferenceImageFilter::InitializeFunctionCoefficients() and<br>
> ImageRegion::IsInside(const Self&) disappear when a local object is<br>
> replaced by a const-reference.<br>
><br>
> In "Core\FiniteDifference\include\itkFiniteDifferenceImageFilter.hxx"<br>
> (line 273):<br>
><br>
>  - const SpacingType spacing = outputImage->GetSpacing();<br>
>  + const SpacingType & spacing = outputImage->GetSpacing();<br>
><br>
><br>
> In "Core\Common\include\itkImageRegion.h" (line 260):<br>
><br>
>  - SizeType&  size = region.GetSize();<br>
>  + const SizeType & size = region.GetSize();<br>
><br>
><br>
> I must admit I don't know why such a modification actually does suppress<br>
> the crashes. But it seems to do the job! What do you think?<br>
><br>
> Kind regards, Niels<br>
><br>
</div></div></blockquote></div><br></div>