<div>Sean,</div>
<div>&nbsp;</div>
<div>Go ahead and check in the fix.</div>
<div>&nbsp;</div>
<div>Thanks for thinking green...</div>
<div>&nbsp;</div>
<div>Bill<br><br>&nbsp;</div>
<div><span class="gmail_quote">On 7/10/07, <b class="gmail_sendername">Sean McBride</b> &lt;<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>&gt; wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">On 7/10/07 10:54 AM, Bill Lorensen said:<br><br>&gt;I notice that the code in ITK&#39;s version of DICOMParser just &quot;break&quot;&#39;s for
<br>&gt;both VR_FL and VR_FD. No swapping is done. I have more confidence in the ITK<br>&gt;implementation than I do in the VTK implementation.<br><br>Bill,<br><br>Sounds good.&nbsp;&nbsp;Should be the same as Mathieu&#39;s previous patch (at
<br>runtime) since the &#39;default&#39; case just breaks too.<br><br>Shall we check this in then?<br><br>&gt;Note though, we<br>&gt;recommend using GDCM in ITK. DICOMParser remains for backward compatibility.<br><br>We are investigating this not because we use it, but because we&#39;re
<br>trying to get the dashboards greener.&nbsp;&nbsp;There are very few Mac VTK<br>dashboards, and almost all of them have failing tests.&nbsp;&nbsp;We actually do<br>our DICOM reading with ITK.<br><br>Thanks,<br><br>--<br>____________________________________________________________
<br>Sean McBride, B. Eng&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>Rogue Research&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="http://www.rogue-research.com">www.rogue-research.com</a><br>
Mac Software Developer&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Montréal, Québec, Canada<br><br><br></blockquote></div><br>