[voikko] enchant diff 17.8.2007

[Harri Pitkänen]

On Friday 17 August 2007 07:14, Hannu Väisänen wrote:
> Muutokset edelliseen versioon:
>
> Merkki ' hyväksytään sanan sisällä.
>
> Hyväksytään "-ddictionary" eikä vain "-d dictionary".
>
>
> Diffi on liitteenä.

Näyttää toimivan. Muutama huomio vielä:



  * Copyright (C) 2003 Dom Lachowicz
+ *               2007 Hannu V�is�nen
  *

Tuossa ä-kirjaimet voisi olla parempi esittää utf8-koodattuna, kuten 
muuallakin enchantin lähdekoodissa



 	fprintf (to, "\t-a lists alternatives.\n");
+	fprintf (to, "\t-d dict uses dictionary <dict>. This flag is for 
emacs/ispell compatibility.\n");
 	fprintf (to, "\t-l lists misspellings.\n");
+	fprintf (to, "\t-m is ignored. This flag is for emacs/ispell compatibility.
\n");
 	fprintf (to, "\t-L displays line numbers.\n");

Emacsin mainitseminen tässä (tai muuallakaan) taitaa olla tarpeetonta, kysehän 
on vain ispell-yhteensopivuudesta. En tiedä, onko sitäkään tarpeen erikseen 
mainita, koska koko ohjelman tarkoitus on olla ispell-yhteensopiva :) 
Valitsinta -L ei muuten ole toteutettu, mutta tämä onkin bugi alkuperäisessä 
ohjelmassa. Sen ehkä voisi poistaa tuosta ohjeestakin, koska oikeassa 
ispellissä -L tekee jotain aivan muuta kuin mitä tuossa väitetään.



-	lang = enchant_get_user_language();
-	if(!lang)
-		return 1;
+	if (dictionary) {
+	        if (!strcmp(dictionary,"suomi") || !strcmp(dictionary,"finnish"))
+		        lang = g_strdup ("fi");  /* For Emacs/Voikko compatibility. */
+		else
+			lang = g_strdup (dictionary);
+	}
+	else {
+	        lang = enchant_get_user_language();
+		if(!lang)
+			return 1;
+ 	}

Muunnokset ispell-kielikoodeista enchant-kielikoodeiksi voisi ehkä tehdä 
omassa funktiossaan, koska niitä tarvitaan jatkossa paljon muitakin 
("american" -> "en_US" jne.) Ehkäpä tämän muunnoksen voisi tehdä 
enchant-kirjaston sisällä, tosin en tiedä, käytetäänkö noita missään muualla 
kuin ispellissä. "suomi" -> "fi" -muunnos on tarpeen vain 
tmispell-yhteensopivuuden takia, mutta kaipa sekin on hyödyllinen asia. 
Näistä asioista kannattaisi varmaan keskustella Dom Lachowiczin kanssa.



Minusta tuota voisi jo tarjota eteen päin. Vaikka vielä tarvitaan muutakin, 
jos enchantia halutaan käyttää esimerkiksi nykyisessä KDE:ssa, niin nämäkin 
muutokset ovat jo aika pitkä askel oikeaan suuntaan, ja ilmeisesti riittävät 
Emacsille.

Harri