Sounds reasonable to me. I recently modified that class to deal with modularization issues and found its contents to be a bit eyebrow raising myself.<br><br><div class="gmail_quote">On Wed, Jun 22, 2011 at 1:14 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com">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;">Hi All,<br>
<br>
I'd like to remove vtkFastNumericConversion.h/.cxx.  This is an<br>
obsolete file uses bit-tricks to accelerate float-to-int conversions<br>
on old pentium processors (i.e. Pentium II and earlier, or any code<br>
compiled for compatibility with such processors).  It has now become<br>
irrelevant and, in my opinion, dangerous.<br>
<br>
Of note is that:<br>
1) It applies these "bit tricks" whenever compiled a 32-bit x86<br>
processor, even though modern i386-family processors do not need them<br>
2) On all other processors, it gives incorrect results!!!<br>
3) It is only used in the ray cast volume mapper, as far as I've seen<br>
<br>
The incorrect results I mentioned are as follows: whenever<br>
VTK_USE_TRICK is not set, it falls back to this code:<br>
<br>
int QuickFloor(double val)<br>
  {<br>
  return static_cast<int>(val);<br>
  }<br>
<br>
int SafeFloor(double val)<br>
  {<br>
  return static_cast<int>(val);<br>
  }<br>
<br>
In other words, on any non-i386 CPU, including x86_64, the results are<br>
incorrect if "val" is negative.  So I would like to mark the whole<br>
thing "LEGACY" for now, and remove the file completely before the VTK<br>
6.0 release.<br>
<br>
 - David<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>