<div dir="ltr">Just so I can keep VTK in sync with GL2PS upstream -- I assume the silence means this patch will not be accepted?<div><br></div><div style>Thanks,</div><div style>Dave</div></div><div class="gmail_extra"><br>

<br><div class="gmail_quote">On Fri, May 10, 2013 at 8:27 AM, David Lonie <span dir="ltr"><<a href="mailto:david.lonie@kitware.com" target="_blank">david.lonie@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

After a good night's sleep and a peek at the OpenGL specification,<br>
this patch may not be needed. The spec states:<br>
<br>
Any representable floating-point value is legal as input to a GL<br>
command that requires floating-point data. The result of providing a<br>
value that is not a floating point number to such a command is<br>
unspecified, but must not lead to GL interruption or termination. In<br>
IEEE arithmetic, for example, providing a negative zero or a<br>
denormalized number to a GL command yields predictable results, while<br>
providing a NaN or an infinity yields unspecified results.<br>
<br>
We may not want to add the overhead of explicitly checking each<br>
coordinate for Nan, when passing Nans in is a user error. GL2PS does<br>
adhere to the standard -- it does not crash or terminate, and rather<br>
gracefully passes the Nans through to the output.<br>
<br>
Feel free to reject this patch, I'm on the fence about it as it.<br>
<br>
Dave<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, May 9, 2013 at 5:14 PM, David Lonie <<a href="mailto:david.lonie@kitware.com">david.lonie@kitware.com</a>> wrote:<br>
> This bit is also needed to get this working on windows.<br>
><br>
> diff --git a/ThirdParty/gl2ps/vtkgl2ps/gl2ps.c<br>
> b/ThirdParty/gl2ps/vtkgl2ps/gl2ps.c<br>
> index 6c98d81..d256bde 100644<br>
> --- a/ThirdParty/gl2ps/vtkgl2ps/gl2ps.c<br>
> +++ b/ThirdParty/gl2ps/vtkgl2ps/gl2ps.c<br>
> @@ -55,6 +55,10 @@<br>
>  #include <vtk_png.h><br>
>  #endif<br>
><br>
> +#ifdef _MSC_VER<br>
> +#define isfinite _finite<br>
> +#endif<br>
> +<br>
>  /*********************************************************************<br>
>   *<br>
>   * Private definitions, data structures and prototypes<br>
><br>
> On Thu, May 9, 2013 at 4:57 PM, David Lonie <<a href="mailto:david.lonie@kitware.com">david.lonie@kitware.com</a>> wrote:<br>
>> Hi list,<br>
>><br>
>> We've run into an issue exporting plots that contain Nan values.<br>
>> OpenGL seems to handle these gracefully on most implementations by<br>
>> discarding the primitive and moving on. The attached patch does the<br>
>> same for GL2PS.<br>
>><br>
>> Can this be included in the next release?<br>
>><br>
>> Thanks,<br>
>><br>
>> Dave<br>
</div></div></blockquote></div><br></div>