FYI, the fix for this from Mathieu has also been applied to the release branch.<br><br><div class="gmail_quote">On Fri, Feb 11, 2011 at 1:58 PM, Mark Roden <span dir="ltr"><<a href="mailto:mmroden@gmail.com">mmroden@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Working now with 886!<br>
<br>
whooo!<br>
<br>
I was getting distracted by a crash from this code:<br>
<br>
<br>
            vtkFileOutputWindow outWin = new vtkFileOutputWindow();<br>
            //outWin.SetInstance(outWin);//vtk 5.9 does not want this to be here<br>
            outWin.SetFileName("MVSVTKViewer.log");<br>
<br>
but commenting out the second line appears to make it all work.  Now<br>
there are a bunch of things that have changed that require fixing...<br>
<br>
Thanks!<br>
<font color="#888888">Mark<br>
</font><div><div></div><div class="h5"><br>
On Thu, Feb 10, 2011 at 3:53 PM, Mark Roden <<a href="mailto:mmroden@gmail.com">mmroden@gmail.com</a>> wrote:<br>
> Hi all,<br>
><br>
> Thanks for your work on this, Mathieu.  Unfortunately, the fix doesn't<br>
> work; it actually breaks vtk's handling of strings elsewhere, before<br>
> my code even gets to the vtkGDCM code.  Everything will compile, but<br>
> it won't run.<br>
><br>
> I've reflected this in the gerrit patch.  Let me know what else I can<br>
> do; I'll definitely keep an eye open for other patches, or can skype<br>
> for fixing issues in a pair.<br>
><br>
> Mark<br>
><br>
> On Thu, Feb 10, 2011 at 5:13 AM, David Partyka<br>
> <<a href="mailto:david.partyka@kitware.com">david.partyka@kitware.com</a>> wrote:<br>
>> When this gets committed let me know and I will put it on the release branch<br>
>> as well.<br>
>><br>
>> On Thu, Feb 10, 2011 at 4:01 AM, Mathieu Malaterre<br>
>> <<a href="mailto:mathieu.malaterre@gmail.com">mathieu.malaterre@gmail.com</a>> wrote:<br>
>>><br>
>>> Mark,<br>
>>><br>
>>>  I believe I found your issue:<br>
>>><br>
>>> <a href="http://review.source.kitware.com/#change,886" target="_blank">http://review.source.kitware.com/#change,886</a><br>
>>><br>
>>> Thanks for reviewing.<br>
>>><br>
>>> On Wed, Feb 9, 2011 at 3:33 AM, Mark Roden <<a href="mailto:mmroden@gmail.com">mmroden@gmail.com</a>> wrote:<br>
>>> > Hi all,<br>
>>> ><br>
>>> > I have a problem using gdcm with vtk 5.9 (both found in their<br>
>>> > respective git trunks) when wrapped in java.  This problem does NOT<br>
>>> > happen with vtk 5.6.<br>
>>> ><br>
>>> > If I wrap everything in java, build using java with release or debug<br>
>>> > on both gdcm and vtk, and on mac or on windows using either gcc or<br>
>>> > visual studio 2008, respectively.  The mac is built using unix<br>
>>> > makefiles, not xcode.<br>
>>> ><br>
>>> > I've checked in some sample code in the gdcm repository that should<br>
>>> > show this behavior (at least, it shows it on mine).  It's a mockup of<br>
>>> > the Medical3.java example that's been modified to use the<br>
>>> > vtkGDCMImageReader object instead of the v16 reader.  The file is<br>
>>> > gdcmImageReadingExample.java.  There is quite a bit of code there, but<br>
>>> > the relevant portion is the gdcmReader.Update() line.<br>
>>> ><br>
>>> > The relevant code looks like (where theSize and filenames have been<br>
>>> > defined previously, the entire code is in the example code):<br>
>>> ><br>
>>> > vtkGDCMImageReader reader = new vtkGDCMImageReader();<br>
>>> ><br>
>>> > vtkStringArray sa = new vtkStringArray();<br>
>>> > for (int j = 0; j < theSize; j ++){<br>
>>> >  sa.InsertNextValue(filenames[j]);<br>
>>> > }<br>
>>> ><br>
>>> > gdcmReader.SetFileNames(sa);<br>
>>> > gdcmReader.Update();<br>
>>> ><br>
>>> > vtkImageData theImageData = gdcmReader.GetOutput();<br>
>>> ><br>
>>> > with vtk version 5.6, theImageData is a valid object.  With vtk<br>
>>> > version 5.7 and on, it is not.<br>
>>> ><br>
>>> > What has changed with the java wrapping?  The same code works fine in<br>
>>> > C++, but not when java wrapped.<br>
>>> ><br>
>>> > Please help.  I thought that moving to vtk 5.9 would solve these<br>
>>> > problems, but apparently not.<br>
>>> ><br>
>>> > Thanks,<br>
>>> > Mark<br>
>>> ><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> Mathieu<br>
>>> _______________________________________________<br>
>>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>><br>
>>> Visit other Kitware open-source projects at<br>
>>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>><br>
>>> Follow this link to subscribe/unsubscribe:<br>
>>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>><br>
>><br>
>><br>
><br>
</div></div></blockquote></div><br>