<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif;">
<div>I’ll have a look when review.source.kitware.com comes back up.</div>
<div><br>
</div>
<div>It would be OK to replaces the [] operator with using the iterator directly. The only reason I can think of not to is that even with a DICOM file, the time spent in writing and reading the MetaDataDictionary is not going to have a significant computational
cost, since it only happens when you read or write a DICOM file. It’s not something done in the inner loop of a big computation.</div>
<div><br>
</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>Arnaud Gelas <<a href="mailto:arnaudgelas@gmail.com">arnaudgelas@gmail.com</a>><br>
<span style="font-weight:bold">Date: </span>Monday, September 15, 2014 at 5:05 AM<br>
<span style="font-weight:bold">To: </span>ITK <<a href="mailto:insight-developers@itk.org">insight-developers@itk.org</a>><br>
<span style="font-weight:bold">Subject: </span>[ITK-dev] help itk::GDCMImageIO / itk::ExposeMetaData<br>
</div>
<div><br>
</div>
<div>
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">
<div>Hi guys,</div>
<div><br>
</div>
<div>I have been looking at GDCMImageIO this morning and I thought it would be great if it was possible to iterate directly on the Dictionary and do not use ExposeMetaData cause internally there are 2 finds (HasKeys and [] ) per iterator (in which we already
know both keys and values).</div>
<div><br>
</div>
<div><a href="https://github.com/Kitware/ITK/blob/master/Modules/IO/GDCM/src/itkGDCMImageIO.cxx#L700">https://github.com/Kitware/ITK/blob/master/Modules/IO/GDCM/src/itkGDCMImageIO.cxx#L700</a></div>
<div><a href="https://github.com/Kitware/ITK/blob/master/Modules/Core/Common/include/itkMetaDataObject.h#L171-L189">https://github.com/Kitware/ITK/blob/master/Modules/Core/Common/include/itkMetaDataObject.h#L171-L189</a></div>
<div><br>
</div>
<div>I have tried to create a new function which would take an iterator as parameter, I have compilation error or a failing dynamic_cast… I guess I am missing something obvious…</div>
<div><br>
</div>
<div>I have been testing something like that:</div>
<div><br>
</div>
<div><a href="http://review.source.kitware.com/#/c/17026/">http://review.source.kitware.com/#/c/17026/</a></div>
<div><br>
</div>
<div>Any idea on how to fix the code ( both compilation & dynamic_cast ), please?</div>
<div><br>
</div>
<div>Thanks,</div>
<div>Arnaud</div>
</div>
</div>
</span><br>
<br>
<hr>
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged. If you are not the intended recipient, you are hereby notified that any
retention, dissemination, distribution, or copying of this communication is strictly prohibited. Please reply to the sender that you have received the message in error, then delete it. Thank you.
<hr>
</body>
</html>