<div dir="ltr">Will do.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 2, 2016 at 1:08 PM, Berk Geveci <span dir="ltr"><<a href="mailto:berk.geveci@kitware.com" target="_blank">berk.geveci@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Ugh. That's a typo (probably copy & paste error). If you look at the implementation, it actually doesn't check for NULL pointer. I am swamped now. Do you have a few cycles to push a fix?</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 2, 2016 at 2:12 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Berk,</div><div><br></div><div>One of the things that I'm not quite sure about is, why is it permitted to call algorithm->UpdateExtent() with no arguments? </div><br><div class="gmail_extra">  int UpdateExtent(const int extents[6]=0)</div><div class="gmail_extra"><br></div><div class="gmail_extra">If I understand the code, UpdateExtent() with no args is the same as Update().  It seems dangerous to allow this, because a programmer could easily type UpdateExtent() by accident instead of UpdateWholeExtent(), especially if they have autocompletion turned on in their code editor (and especially because Update() and UpdateWholeExtent() give the same result most of the time).</div><span><font color="#888888"><div class="gmail_extra"><br></div><div class="gmail_extra"> - David</div></font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>