Dramatically improve windows session-save-time by avoiding recursive calls to set_locale.

1 files changed, 17 insertions, 12 deletions

diff --git a/libs/pbd/locale_guard.cc b/libs/pbd/locale_guard.cc
index 44b9fc2b64..b8493cf176 100644
--- a/libs/pbd/locale_guard.cc
+++ b/libs/pbd/locale_guard.cc
@@ -25,24 +25,29 @@
 
 using namespace PBD;
 
+// try to avoid calling setlocale() recursively.  this is not thread-safe.
+std::string PBD::LocaleGuard::current;
+
 LocaleGuard::LocaleGuard (const char* str)
+ : old(0)
 {
-	old = setlocale (LC_NUMERIC, NULL);
-
-        if (old) {
-                old = strdup (old);
-                if (strcmp (old, str)) {
-                        setlocale (LC_NUMERIC, str);
-                }
-        }
+	if (current != str) {
+		old = strdup (setlocale (LC_NUMERIC, NULL));
+		if (strcmp (old, str)) {
+			if (setlocale (LC_NUMERIC, str))
+				current = str; 
+		}
+	}
 }
 
 LocaleGuard::~LocaleGuard ()
 {
-	setlocale (LC_NUMERIC, old);
+	if (old) {
+		if (setlocale (LC_NUMERIC, old))
+			current = old;
 
-        if (old) {
-                free (const_cast<char*>(old));
-        }
+		free ((char*)old);
+	}
 }
 
+