<div dir="ltr">+1</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 22, 2013 at 6:31 PM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi again,<br>
<br>
Looking in vtkType.h we see things like VTK_INT_MIN and VTK_INT_MAX and lots of other MIN and MAX varieties.<br>
<br>
I came across this snippit: "-VTK_LARGE_ID-1" and thought, 'Why didn't he use the MIN variant?'  Well, there isn't one.<br>
<br>
Anyone object to:<br>
 - a rename of VTK_LARGE_ID to VTK_ID_MAX<br>
 - a find-replace of the 58 instances of VTK_LARGE_ID to VTK_ID_MAX<br>
 - the creation of VTK_ID_MIN<br>
 - a find-replace of the 2 instances of -VTK_LARGE_ID-1 to VTK_ID_MIN<br>
 - adding VTK_LARGE_ID in VTK_LEGACY_REMOVE for backwards compatibility<br>
<br>
I think it would improve consistency and readability.<br>
<br>
Cheers,<br>
<br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br></div>