diff --git a/docs/changes.txt b/docs/changes.txt index fc7fcb1e00..c3d49daf94 100644 --- a/docs/changes.txt +++ b/docs/changes.txt @@ -108,6 +108,10 @@ Changes in behaviour which may result in build errors 3.1.3: (released 2019-??-??) ---------------------------- +All: + +- Fix regression in wxTranslations::AddCatalog() in 3.1.2 (Tomasz Słodkowicz). + All (GUI): - Fix wxInfoBar close button size in high DPI (Stefan Ziegler). diff --git a/src/common/translation.cpp b/src/common/translation.cpp index 7d9937cd18..26515b4c9b 100644 --- a/src/common/translation.cpp +++ b/src/common/translation.cpp @@ -1574,18 +1574,18 @@ bool wxTranslations::AddCatalog(const wxString& domain, wxS("adding '%s' translation for domain '%s' (msgid language '%s')"), *lang, domain, msgIdLang); - // No use loading languages that are less preferred than the - // msgid language, as by definition it contains all the strings - // in the msgid language. - if ( msgIdLang == *lang ) - break; - // We determine success by the success of loading/failing to load // the most preferred (i.e. the first one) language's catalog: if ( lang == domain_langs.begin() ) success = LoadCatalog(domain, *lang, msgIdLang); else LoadCatalog(domain, *lang, msgIdLang); + + // No use loading languages that are less preferred than the + // msgid language, as by definition it contains all the strings + // in the msgid language. + if ( msgIdLang == *lang ) + break; } return success;