milliams wrote:
I've been seeing those warnings on my own builds in Linux since the beginning so I'm not sure why they weren't showing up before. I would say that it's correct that they do show up now even though they're not that useful. It doesn't look like it's possible to explicitly disable just those warnings though.
I've renamed all the files called glew.c to glew.cpp. I'm not sure if this has any other knock-on effects as to how the compiler treats the files but it seems to work ok, and it's only for the examples anyway. It's either that or we find a way to make CMake pass different command line parameters to different files. This is untested on Linux, so let's see if the warnings are gone tonight.
milliams wrote:
I did notice the failing AmbientOcclusionCalculator test and I saw that you changed the first of the comparisons to be equal to the Linux result. Now it seems to be complaining about the second one. On my Linux box (similar GCC version but everything else is different) I get the same results as the build box so it seems consistent on Linux.
Yes, so apparently the second one passed the first time. I'm not sure it's entirely consistant even on one machine. I've updated it anyway, and we'll see if it fails again. And at some point I'll try to make it match on Windows.