#include "LyXRC.h"
#include "WordLangTuple.h"
+#include "frontends/alert.h"
+
#include "support/debug.h"
#include "support/docstring_list.h"
+#include "support/filetools.h"
#include "support/FileName.h"
#include "support/gettext.h"
#include "support/lassert.h"
+#include "support/lstrings.h"
#include "support/os.h"
#include <hunspell/hunspell.hxx>
#include <map>
#include <string>
+#include <vector>
using namespace std;
using namespace lyx::support;
namespace {
typedef map<std::string, Hunspell *> Spellers;
+typedef vector<WordLangTuple> IgnoreList;
} // anon namespace
~Private();
+ bool haveDictionary(string const & lang, string & hpath);
Hunspell * addSpeller(string const & lang);
Hunspell * speller(string const & lang);
+ /// ignored words
+ bool isIgnored(WordLangTuple const & wl) const;
/// the spellers
Spellers spellers_;
+ ///
+ IgnoreList ignored_;
};
}
-Hunspell * HunspellChecker::Private::addSpeller(string const & lang)
+namespace {
+bool haveLanguageFiles(string const & hpath)
{
- string hunspell_path = external_path(lyxrc.hunspelldir_path);
- LYXERR(Debug::FILES, "hunspell path: " << hunspell_path);
- if (hunspell_path.empty())
- return false;
-
- hunspell_path += "/" + lang;
- // replace '_' with '-' as this is the convention used by hunspell.
- hunspell_path[hunspell_path.size() - 3] = '-';
- FileName const affix(hunspell_path + ".aff");
- FileName const dict(hunspell_path + ".dic");
+ FileName const affix(hpath + ".aff");
+ FileName const dict(hpath + ".dic");
if (!affix.isReadableFile()) {
// FIXME: We should indicate somehow that this language is not
// supported.
LYXERR(Debug::FILES, "Hunspell affix file " << affix << " does not exist");
- return 0;
+ return false;
}
if (!dict.isReadableFile()) {
LYXERR(Debug::FILES, "Hunspell dictionary file " << dict << " does not exist");
- return 0;
+ return false;
}
+ return true;
+}
+}
+
+
+bool HunspellChecker::Private::haveDictionary(string const & lang, string & hunspell_path)
+{
+ LYXERR(Debug::FILES, "hunspell path: " << external_path(hunspell_path));
+ if (hunspell_path.empty()) {
+ // FIXME We'd like to issue a better error message here, but there seems
+ // to be a problem about thread safety, or something of the sort. If
+ // we issue the message using frontend::Alert, then the code comes
+ // back through here while the box is waiting, and causes some kind
+ // of crash.
+ static bool warned = false;
+ if (!warned) {
+ warned = true;
+ LYXERR0("Hunspell path not set.");
+ //frontend::Alert::error(_("Hunspell Path Not Found"),
+ // _("You must set the Hunspell dictionary path in Tools>Preferences>Paths."));
+ }
+ return false;
+ }
+
+ hunspell_path = external_path(addName(hunspell_path, lang));
+ if (!haveLanguageFiles(hunspell_path)) {
+ // try with '_' replaced by '-'
+ hunspell_path = subst(hunspell_path, '_', '-');
+ if (!haveLanguageFiles(hunspell_path)) {
+ // FIXME: We should indicate somehow that this language is not
+ // supported, probably by popping a warning. But we'll need to
+ // remember which warnings we've issued.
+ return false;
+ }
+ }
+ return true;
+}
+
+
+Hunspell * HunspellChecker::Private::addSpeller(string const & lang)
+{
+ string hunspell_path = lyxrc.hunspelldir_path;
+
+ if (!haveDictionary(lang, hunspell_path))
+ return 0;
+
+ FileName const affix(hunspell_path + ".aff");
+ FileName const dict(hunspell_path + ".dic");
Hunspell * h = new Hunspell(affix.absFilename().c_str(), dict.absFilename().c_str());
spellers_[lang] = h;
return h;
}
+bool HunspellChecker::Private::isIgnored(WordLangTuple const & wl) const
+{
+ IgnoreList::const_iterator it = ignored_.begin();
+ for (; it != ignored_.end(); ++it) {
+ if ((*it).lang()->code() != wl.lang()->code())
+ continue;
+ if ((*it).word() == wl.word())
+ return true;
+ }
+ return false;
+}
+
+
HunspellChecker::HunspellChecker(): d(new Private)
{
}
SpellChecker::Result HunspellChecker::check(WordLangTuple const & wl)
{
- string const word_to_check = to_utf8(wl.word());
- Hunspell * h = d->speller(wl.lang_code());
+ if (d->isIgnored(wl))
+ return OK;
+
+ Hunspell * h = d->speller(wl.lang()->code());
if (!h)
return OK;
int info;
+
+ string const encoding = h->get_dic_encoding();
+ string const word_to_check = to_iconv_encoding(wl.word(), encoding);
+
if (h->spell(word_to_check.c_str(), &info))
return OK;
void HunspellChecker::insert(WordLangTuple const & wl)
{
string const word_to_check = to_utf8(wl.word());
- Hunspell * h = d->speller(wl.lang_code());
+ Hunspell * h = d->speller(wl.lang()->code());
if (!h)
return;
h->add(word_to_check.c_str());
}
-void HunspellChecker::accept(WordLangTuple const & word)
+void HunspellChecker::accept(WordLangTuple const & wl)
{
- // FIXME: not implemented!
+ d->ignored_.push_back(wl);
}
docstring_list & suggestions)
{
suggestions.clear();
- string const word_to_check = to_utf8(wl.word());
- Hunspell * h = d->speller(wl.lang_code());
+ Hunspell * h = d->speller(wl.lang()->code());
if (!h)
return;
- char *** suggestion_list = 0;
+ string const encoding = h->get_dic_encoding();
+ string const word_to_check = to_iconv_encoding(wl.word(), encoding);
+ char ** suggestion_list;
+ int const suggestion_number = h->suggest(&suggestion_list, word_to_check.c_str());
+ if (suggestion_number <= 0)
+ return;
+ for (int i = 0; i != suggestion_number; ++i)
+ suggestions.push_back(from_iconv_encoding(suggestion_list[i], encoding));
+ h->free_list(&suggestion_list, suggestion_number);
+}
- // FIXME: Hunspell::suggest() crashes on Win/MSVC9
- return;
- int const suggestion_number = h->suggest(suggestion_list, word_to_check.c_str());
- if (suggestion_number == 0)
- return;
- h->free_list(suggestion_list, suggestion_number);
+bool HunspellChecker::hasDictionary(Language const * lang) const
+{
+ if (!lang)
+ return false;
+ string hunspell_path = lyxrc.hunspelldir_path;
+ return (d->haveDictionary(lang->code(), hunspell_path));
}