Back in March[1], I sent a patch porting _mesa_strto[df] to C rather than C++. I fixed up the patch according to the criticism, but unfortunately I dropped the ball before I sent out the result. So here I am, picking it back up!
This time I've taken a deeper dive into the whole init/deinit of Mesa, and cleaned up a bunch of stuff in that area. And as a result, this time we end up freeing the locale also. No Piglit regressions observed. [1]: <1426446329-23984-1-git-send-email-kusmab...@gmail.com> Erik Faye-Lund (7): mesa/main: Get rid of outdated GDB-hack dri: don't touch the shader compiler mesa/main: only call _mesa_destroy_shader_compiler once on exit glsl: No need to lock in _mesa_glsl_release_types util: port _mesa_strto[df] to C mesa/main: free locale at exit util: assert to verify that locale is initialized src/glsl/glcpp/glcpp.c | 3 ++ src/glsl/glsl_types.cpp | 8 +-- src/glsl/main.cpp | 3 ++ src/mesa/drivers/dri/common/dri_util.c | 3 -- src/mesa/main/context.c | 47 ++++++------------ src/util/Makefile.sources | 2 +- src/util/strtod.c | 91 ++++++++++++++++++++++++++++++++++ src/util/strtod.cpp | 75 ---------------------------- src/util/strtod.h | 6 +++ 9 files changed, 123 insertions(+), 115 deletions(-) create mode 100644 src/util/strtod.c delete mode 100644 src/util/strtod.cpp -- 2.1.4 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev