<div dir="ltr">That sounds reasonable to me.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 12, 2015 at 8:33 AM, Xabi Riobe <span dir="ltr"><<a href="mailto:xabivtk@gmail.com" target="_blank">xabivtk@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">Hi,<div><br></div><div>I see in the code of vtkImageData::GetDimensions(int dims[3]) that the "Dimensions" member variable is not computed anymore since commit <a href="https://gitlab.kitware.com/vtk/vtk/commit/4345f286e7fbffe0d8734ade60824556cb1b3d3d" style="color:rgb(51,60,72);text-decoration:none;font-family:'Source Sans Pro','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:15px;line-height:21.4286px" target="_blank">4345f286e7fbffe0d8734ade60824556cb1b3d3d</a>, for thread safe reasons.</div><div><br></div><div>I can understand the reason for this change if it can not be done otherway, but i think the documentation of the method should be updated, since it is curently : </div><div>"Dimensions are computed from Extents during this call."</div><div><br></div><div>It is true for the int *GetDimensions() implementation but not this one anymore.</div><div><br></div><div>Cheers,</div><div>Xabi</div></div>
<br>_______________________________________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Cory Quammen<br>R&D Engineer<br>Kitware, Inc.</div>
</div>