Thanks for the reviews! <br><br><div class="gmail_quote">On Sat, Dec 8, 2012 at 9:54 PM, Steve M. Robbins <span dir="ltr">&lt;<a href="mailto:steve@sumost.ca" target="_blank">steve@sumost.ca</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello Matt,<br>
<br>
Did not compile with patches, but review comments follow.<br>
<div class="im"><br>
On December 7, 2012 10:28:01 AM Matt McCormick wrote:<br>
&gt; Hi Steve,<br>
&gt;<br>
&gt; Thanks again for you <a href="http://riemann.sumost.ca" target="_blank">riemann.sumost.ca</a> dashboard submissions.<br>
&gt;<br>
&gt; I created patches to address warnings on that box:<br>
&gt;<br>
&gt;   <a href="http://review.source.kitware.com/#/c/8846/" target="_blank">http://review.source.kitware.com/#/c/8846/</a><br>
<br>
</div>Looks fine.<br>
<br>
&gt;   <a href="http://review.source.kitware.com/#/c/8847/" target="_blank">http://review.source.kitware.com/#/c/8847/</a><br>
<br>
While I believe the change is correct, I would have used<br>
IndexType::IndexValueType (for x, y) rather than itk::IndexValueType, to make<br>
the subsequent line<br>
<br>
    const IndexType index = { { x, y } };<br>
<br>
more clearly correct.<br>
<br>
Cheers,<br>
-Steve<br>
<br>
</blockquote></div><br>