On 11 June 2013 09:52, Stefan Dösinger <stefandoesin...@gmail.com> wrote: > I think this should be added as a code comment above shader_glsl_ftoa, I > don't think the reason why we need our own conversion function is obvious. > I originally considered adding a comment, but figured it was obvious enough from being in the context of GLSL shader generation. I don't feel particularly strongly about it though, so I can add it again if it helps.
> The ARB shader backend will need this as well, but I can take care of this. > What external software triggers this behavior? > I'm not sure, this was reported on IRC. But you can easily reproduce it by e.g. adding setlocale(LC_ALL, ""); to wined3d_dll_init() and running the tests with an appropriate locale set. > shader_glsl_ftoa doesn't handle inf or nan. I assume that's intentional > because they would only trigger a parser error. > Yeah, inf and nan aren't really meaningful here. If we really wanted to it would only be a couple of extra lines though.