X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Faspell.C;h=69252704041db9471ae5ec63dd67acad668c261d;hb=c727c6db7f2dd7f6a727462f5d11964888d0e76e;hp=507437c6339e1d1dc6f7b2703977a4db45b05537;hpb=46756fd9e4087fd30d9de504684d83803cd4b4c5;p=lyx.git

diff --git a/src/aspell.C b/src/aspell.C
index 507437c633..6925270404 100644
--- a/src/aspell.C
+++ b/src/aspell.C
@@ -11,8 +11,6 @@
 
 #include <config.h>
 
-#ifdef USE_ASPELL
-
 #include "debug.h"
 
 #include <aspell.h>
@@ -24,6 +22,7 @@
 
 using std::string;
 
+namespace lyx {
 
 ASpell::ASpell(BufferParams const &, string const & lang)
 	: els(0), spell_error_object(0)
@@ -56,7 +55,16 @@ ASpell::~ASpell()
 void ASpell::addSpeller(string const & lang)
 {
 	AspellConfig * config = new_aspell_config();
+	// FIXME The aspell documentation says to use "lang"
 	aspell_config_replace(config, "language-tag", lang.c_str());
+	// Set the encoding to utf-8.
+	// aspell does also understand "ucs-4", so we would not need a
+	// conversion in theory, but if this is used it expects all
+	// char const * arguments to be a cast from  uint const *, and it
+	// seems that this uint is not compatible with our char_type on some
+	// platforms (cygwin, OS X). Therefore we use utf-8, that does
+	// always work.
+	aspell_config_replace(config, "encoding", "utf-8");
 	AspellCanHaveError * err = new_aspell_speller(config);
 	if (spell_error_object)
 		delete_aspell_can_have_error(spell_error_object);
@@ -75,7 +83,7 @@ void ASpell::addSpeller(string const & lang)
 
 ASpell::Result ASpell::check(WordLangTuple const & word)
 {
-	Result res = UNKNOWN;
+	Result res = UNKNOWN_WORD;
 
 	Spellers::iterator it = spellers_.find(word.lang_code());
 	if (it == spellers_.end()) {
@@ -88,20 +96,20 @@ ASpell::Result ASpell::check(WordLangTuple const & word)
 
 	AspellSpeller * m = it->second.speller;
 
-	int const word_ok = aspell_speller_check(m, word.word().c_str(), -1);
+	int const word_ok = aspell_speller_check(m, to_utf8(word.word()).c_str(), -1);
 	BOOST_ASSERT(word_ok != -1);
 
 	if (word_ok) {
 		res = OK;
 	} else {
 		AspellWordList const * sugs =
-			aspell_speller_suggest(m, word.word().c_str(), -1);
+			aspell_speller_suggest(m, to_utf8(word.word()).c_str(), -1);
 		BOOST_ASSERT(sugs != 0);
 		els = aspell_word_list_elements(sugs);
 		if (aspell_word_list_empty(sugs))
-			res = UNKNOWN;
+			res = UNKNOWN_WORD;
 		else
-			res = MISSED;
+			res = SUGGESTED_WORDS;
 	}
 	return res;
 }
@@ -111,7 +119,7 @@ void ASpell::insert(WordLangTuple const & word)
 {
 	Spellers::iterator it = spellers_.find(word.lang_code());
 	if (it != spellers_.end())
-		aspell_speller_add_to_personal(it->second.speller, word.word().c_str(), -1);
+		aspell_speller_add_to_personal(it->second.speller, to_utf8(word.word()).c_str(), -1);
 }
 
 
@@ -119,30 +127,32 @@ void ASpell::accept(WordLangTuple const & word)
 {
 	Spellers::iterator it = spellers_.find(word.lang_code());
 	if (it != spellers_.end())
-		aspell_speller_add_to_session(it->second.speller, word.word().c_str(), -1);
+		aspell_speller_add_to_session(it->second.speller, to_utf8(word.word()).c_str(), -1);
 }
 
 
-string const ASpell::nextMiss()
+docstring const ASpell::nextMiss()
 {
 	char const * str = 0;
 
 	if (els)
 		str = aspell_string_enumeration_next(els);
 
-	return (str ? str : "");
+	return (str ? from_utf8(str) : docstring());
 }
 
 
-string const ASpell::error()
+docstring const ASpell::error()
 {
 	char const * err = 0;
-
+	
 	if (spell_error_object && aspell_error_number(spell_error_object) != 0) {
 		err = aspell_error_message(spell_error_object);
 	}
 
-	return (err ? err : "");
+	// FIXME UNICODE: err is not in UTF8, but probably the locale encoding
+	return (err ? from_utf8(err) : docstring());
 }
 
-#endif // USE_ASPELL
+
+} // namespace lyx