fixup!

Gnucash · Sep 12, 2024 · ead1fe8 · ead1fe8
1 parent 941c365
commit ead1fe8
Showing 1 changed file with 69 additions and 50 deletions.
diff --git a/libgnucash/engine/gnc-datetime.cpp b/libgnucash/engine/gnc-datetime.cpp
@@ -75,6 +75,8 @@ static const TZ_Ptr utc_zone(new boost::local_time::posix_time_zone("UTC-0"));
 void _set_tzp(TimeZoneProvider& tz);
 void _reset_tzp();
 
+static boost::gregorian::date gregorian_date_from_locale_string (const std::string& str);
+
 /* To ensure things aren't overly screwed up by setting the nanosecond clock for boost::date_time. Don't do it, though, it doesn't get us anything and slows down the date/time library. */
 #ifndef BOOST_DATE_TIME_HAS_NANOSECONDS
 static constexpr auto ticks_per_second = INT64_C(1000000);
@@ -83,55 +85,6 @@ static constexpr auto ticks_per_second = INT64_C(1000000000);
 #endif
 
 
-static icu::Locale
-get_LCTIME_locale()
-{
-    const char* lc_time_locale = setlocale(LC_TIME, nullptr);
-
-    if (lc_time_locale == nullptr)
-        return icu::Locale::getDefault();
-
-    std::string localeStr(lc_time_locale);
-    size_t dotPos = localeStr.find('.');
-    if (dotPos != std::string::npos)
-        localeStr = localeStr.substr(0, dotPos);
-
-    return icu::Locale::createCanonical (localeStr.c_str());
-}
-
-static Date
-icu_string_to_date (const std::string str)
-{
-    UErrorCode status = U_ZERO_ERROR;
-    icu::Locale locale = get_LCTIME_locale();
-
-    static std::unique_ptr<icu::DateFormat> formatter(icu::DateFormat::createDateInstance(icu::DateFormat::kDefault, locale));
-    if (formatter == nullptr)
-        throw std::invalid_argument ("Cannot create date formatter.");
-
-    static std::unique_ptr<icu::Calendar> calendar(icu::Calendar::createInstance(locale, status));
-    if (U_FAILURE(status))
-        throw std::invalid_argument("Cannot create calendar instance.");
-    calendar->setLenient (false);
-
-    icu::UnicodeString input = icu::UnicodeString::fromUTF8(str);
-    icu::ParsePosition parsePos;
-
-    UDate date = formatter->parse(input, parsePos);
-    if (parsePos.getErrorIndex() != -1 || parsePos.getIndex() != input.length())
-        throw std::invalid_argument ("Cannot parse string");
-
-    calendar->setTime(date, status);
-    if (U_FAILURE(status))
-        throw std::invalid_argument ("Cannot set calendar time");
-
-    return Date (calendar->get(UCAL_YEAR, status),
-                 calendar->get(UCAL_MONTH, status) + 1,
-                 calendar->get(UCAL_DATE, status));
-}
-
-
-
 /* Vector of date formats understood by gnucash and corresponding
  * boost string->date or regex to parse each from an external source
  * Note: while the format names are using a "-" as separator, the
@@ -142,7 +95,7 @@ const std::vector<GncDateFormat> GncDate::c_formats ({
     GncDateFormat { N_("UK date"), boost::gregorian::from_uk_string },
     GncDateFormat { N_("US date"), boost::gregorian::from_us_string },
     GncDateFormat { N_("ISO date"), boost::gregorian::from_string },
-    GncDateFormat { N_("Locale"), icu_string_to_date },
+    GncDateFormat { N_("Locale"), gregorian_date_from_locale_string },
     GncDateFormat {
         N_("y-m-d"),
         "(?:"                                   // either y-m-d
@@ -666,6 +619,72 @@ GncDateTimeImpl::timestamp()
     return str.substr(0, 8) + str.substr(9, 15);
 }
 
+
+
+
+struct ICUResources
+{
+    std::unique_ptr<icu::DateFormat> formatter;
+    std::unique_ptr<icu::Calendar> calendar;
+};
+
+static ICUResources&
+get_icu_resources()
+{
+    static ICUResources rv;
+
+    if (!rv.formatter)
+    {
+        icu::Locale locale;
+
+        const char* lc_time_locale = setlocale(LC_TIME, nullptr);
+        if (lc_time_locale != nullptr)
+        {
+            std::string localeStr(lc_time_locale);
+            size_t dotPos = localeStr.find('.');
+            if (dotPos != std::string::npos)
+                localeStr = localeStr.substr(0, dotPos);
+
+            locale = icu::Locale::createCanonical (localeStr.c_str());
+        }
+
+        rv.formatter.reset(icu::DateFormat::createDateInstance(icu::DateFormat::kDefault, locale));
+        if (!rv.formatter)
+            throw std::invalid_argument("Cannot create date formatter.");
+
+        UErrorCode status = U_ZERO_ERROR;
+        rv.calendar.reset(icu::Calendar::createInstance(locale, status));
+        if (U_FAILURE(status))
+            throw std::invalid_argument("Cannot create calendar instance.");
+
+        rv.calendar->setLenient(false);
+    }
+
+    return rv;
+}
+
+boost::gregorian::date
+static gregorian_date_from_locale_string (const std::string& str)
+{
+    ICUResources& resources = get_icu_resources();
+
+    icu::UnicodeString input = icu::UnicodeString::fromUTF8(str);
+    icu::ParsePosition parsePos;
+
+    UDate date = resources.formatter->parse(input, parsePos);
+    if (parsePos.getErrorIndex() != -1 || parsePos.getIndex() != input.length())
+        throw std::invalid_argument ("Cannot parse string");
+
+    UErrorCode status = U_ZERO_ERROR;
+    resources.calendar->setTime(date, status);
+    if (U_FAILURE(status))
+        throw std::invalid_argument ("Cannot set calendar time");
+
+    return boost::gregorian::date (resources.calendar->get(UCAL_YEAR, status),
+                                   resources.calendar->get(UCAL_MONTH, status) + 1,
+                                   resources.calendar->get(UCAL_DATE, status));
+}
+
 /* Member function definitions for GncDateImpl.
  */
 GncDateImpl::GncDateImpl(const std::string str, const std::string fmt) :