]> git.lyx.org Git - features.git/blobdiff - src/support/Messages.cpp
A first batch of potential bugs spotted by llvm/clang
[features.git] / src / support / Messages.cpp
index b9a5bb5545cbb1436ef983eb882ce2bd7683fac4..4e58a873fd23ac15e60932b29a5e375a376e0d19 100644 (file)
@@ -2,7 +2,7 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author Lars Gullik Bjønnes
+ * \author Lars Gullik Bjønnes
  *
  * Full author contact details are available in file CREDITS.
  */
@@ -14,6 +14,7 @@
 #include "support/debug.h"
 #include "support/docstring.h"
 #include "support/environment.h"
+#include "support/lstrings.h"
 #include "support/Package.h"
 #include "support/unicode.h"
 
 
 #include <cerrno>
 
-using namespace std;
+#  define N_(str) (str)              // for marking strings to be translated
 
-namespace {
+using namespace std;
 
-using lyx::docstring;
-using lyx::from_ascii;
+namespace lyx {
 
 void cleanTranslation(docstring & trans) 
 {
@@ -48,7 +48,7 @@ void cleanTranslation(docstring & trans)
        }
 }
 
-}
+} // lyx
 
 
 #ifdef ENABLE_NLS
@@ -60,7 +60,7 @@ void cleanTranslation(docstring & trans)
 #  if HAVE_GETTEXT
 #    include <libintl.h>      // use the header already in the system *EK*
 #  else
-#    include "../../intl/libintl.h"
+#    include "intl/libintl.h"
 #  endif
 
 using namespace lyx::support;
@@ -69,12 +69,12 @@ namespace lyx {
 
 // This version use the traditional gettext.
 Messages::Messages(string const & l)
-       : lang_(l), warned_(false)
+       : lang_(l)
 {
        // strip off any encoding suffix, i.e., assume 8-bit po files
        size_t i = lang_.find(".");
        lang_ = lang_.substr(0, i);
-       LYXERR(Debug::DEBUG, "language(" << lang_ << ")");
+       LYXERR(Debug::LOCALE, "language(" << lang_ << ")");
 }
 
 
@@ -85,13 +85,13 @@ void Messages::init()
        char const * c = bindtextdomain(PACKAGE, locale_dir.c_str());
        int e = errno;
        if (e) {
-               LYXERR(Debug::DEBUG, "Error code: " << errno << '\n'
-                       << "Directory : " << package().locale_dir().absFilename() << '\n'
+               LYXERR(Debug::LOCALE, "Error code: " << errno << '\n'
+                       << "Directory : " << package().locale_dir().absFileName() << '\n'
                        << "Rtn value : " << c);
        }
 
        if (!bind_textdomain_codeset(PACKAGE, ucs4_codeset)) {
-               LYXERR(Debug::DEBUG, "Error code: " << errno << '\n'
+               LYXERR(Debug::LOCALE, "Error code: " << errno << '\n'
                        << "Codeset   : " << ucs4_codeset);
        }
 
@@ -99,6 +99,70 @@ void Messages::init()
 }
 
 
+string Messages::language() const
+{
+       // get the language from the gmo file
+       string const test = N_("[[Replace with the code of your language]]");
+       string const trans = to_utf8(get(test));
+       if (trans == test) {
+               LYXERR0("Something is weird.");
+               return string();
+       } else
+               return trans;
+}
+
+
+bool Messages::available(string const & c)
+{
+       static string locale_dir = package().locale_dir().toFilesystemEncoding();
+       string code = c;
+       // this loops at most twice
+       while (true) {
+               string const filen = locale_dir + "/" + code 
+                       + "/LC_MESSAGES/" PACKAGE ".mo";
+               if (FileName(filen).isReadableFile())
+                       return true;
+               if (contains(code, '_'))
+                       code = token(code, '_', 0);
+               else return false;
+       }
+       return false;
+
+}
+
+namespace {
+
+// Trivial wrapper around gettext()
+docstring const getText(string const & m)
+{
+       // FIXME: gettext sometimes "forgets" the ucs4_codeset we set
+       // in init(), which leads to severe message corruption (#7371)
+       // We set it again here unconditionally. A real fix must be found!
+       LATTEST(bind_textdomain_codeset(PACKAGE, ucs4_codeset));
+
+       char const * m_c = m.c_str();
+       char const * trans_c = gettext(m_c);
+       docstring trans;
+       if (!trans_c) {
+               LYXERR(Debug::LOCALE, "Undefined result from gettext for `" << m << "'.");
+               trans = from_ascii(m);
+       } else if (trans_c == m_c) {
+               //LYXERR(Debug::LOCALE, "Same as entered returned");
+               trans = from_ascii(m);
+       } else {
+               //LYXERR(Debug::LOCALE, "We got a translation");
+               // m is actually not a char const * but ucs4 data
+               trans = reinterpret_cast<char_type const *>(trans_c);
+       }
+
+       cleanTranslation(trans);
+
+       return trans;
+}
+
+}
+
+
 docstring const Messages::get(string const & m) const
 {
        if (m.empty())
@@ -110,59 +174,37 @@ docstring const Messages::get(string const & m) const
                return it->second;
 
        // The string was not found, use gettext to generate it
-
-       static string const oldLANGUAGE = getEnv("LANGUAGE");
-       static string const oldLC_ALL = getEnv("LC_ALL");
+       docstring trans;
        if (!lang_.empty()) {
                // This GNU extension overrides any language locale
                // wrt gettext.
-               bool success = setEnv("LANGUAGE", lang_);
-               if (!success)
-                       LYXERR0("WARNING: Cannot set env LANGUAGE variable to " << lang_);
+               LYXERR(Debug::LOCALE, "Setting LANGUAGE to " << lang_);
+               EnvChanger language_chg("LANGUAGE", lang_);
                // However, setting LANGUAGE does nothing when the
                // locale is "C". Therefore we set the locale to
                // something that is believed to exist on most
                // systems. The idea is that one should be able to
                // load German documents even without having de_DE
                // installed.
-               success = setEnv("LC_ALL", "en_US");
-               if (!success)
-                       LYXERR0("WARNING: Cannot set env LC_ALL variable to en_US");
+               LYXERR(Debug::LOCALE, "Setting LC_ALL to en_US");
+               EnvChanger lc_all_chg("LC_ALL", "en_US");
 #ifdef HAVE_LC_MESSAGES
                setlocale(LC_MESSAGES, "");
 #endif
-       }
-
-       char const * m_c = m.c_str();
-       char const * trans_c = gettext(m_c);
-       docstring trans;
-       if (!trans_c)
-               LYXERR0("Undefined result from gettext");
-       else if (trans_c == m_c) {
-               LYXERR(Debug::DEBUG, "Same as entered returned");
-               trans = from_ascii(m);
-       } else {
-               LYXERR(Debug::DEBUG, "We got a translation");
-               // m is actually not a char const * but ucs4 data
-               trans = reinterpret_cast<char_type const *>(trans_c);
-       }
-
-       cleanTranslation(trans);
+               trans = getText(m);
+       } else
+               trans = getText(m);
+               
 
-       // Reset environment variables as they were.
-       if (!lang_.empty()) {
-               // Reset everything as it was.
-               setEnv("LANGUAGE", oldLANGUAGE);
-               setEnv("LC_ALL", oldLC_ALL);
 #ifdef HAVE_LC_MESSAGES
-               setlocale(LC_MESSAGES, "");
+       setlocale(LC_MESSAGES, "");
 #endif
-       }
 
+       // store translation in cache
        pair<TranslationCache::iterator, bool> result =
                cache_.insert(make_pair(m, trans));
 
-       LASSERT(result.second, /**/);
+       LASSERT(result.second, return from_utf8(m));
 
        return result.first->second;
 }
@@ -174,7 +216,7 @@ docstring const Messages::get(string const & m) const
 
 namespace lyx {
 
-Messages::Messages(string const & l) {}
+Messages::Messages(string const & /* l */) {}
 
 void Messages::init()
 {
@@ -188,6 +230,16 @@ docstring const Messages::get(string const & m) const
        return trans;
 }
 
+std::string Messages::language() const
+    {
+        return string();
+    }
+
+bool Messages::available(string const & /* c */)
+{
+       return false;
+}
+
 } // namespace lyx
 
 #endif