[voikko-devel] Fwd: Bugi: voikko_remove_extra_hyphenations

[Harri Pitkänen]

On Sunday 25 February 2007 19:55, Kalle Lampila wrote:
> Moi,
>
> Huomasin, että voikko_remove_extra_hyphenations funktio toimii eri
> tavalla kuin sitä ohjaava parametrin toiminta on kommentoitu.
> ...
> Enpä tiedä pitäisikö parametrin kommentti muutaa vastaamaan toimintaa
> vai muuttaa vertailu voikko_remove_extra_hyphenations:
> "if (current_parts > intersect_compound_level)"
> muotoon:
> "if (current_parts > min_parts)"

Hyvä huomio. Koodi tosiaan näyttäisi olevan väärin, pitäisi olla
if (current_parts > min_parts)

Tämä on tietysti jäänyt huomiotta, koska ko. optiota ei varmaan koskaan ole 
testattu muulla kuin oletusarvollaan (jolla se sattuu toimimaan oikein). Edes 
voikkohyphenate ei salli option arvon vaihtamista, eikä sitä ole tiettävästi 
käytetty missään todellisessa sovelluksessa. Pitää lisätä 
voikkohyphenate-ohjelmaan tuo asetus ja laittaa asiaan liittyviä testejä 
mukaan automaattisiin testeihin.

Ihan mielenkiinnosta: oletko kehittämässä jotain sovellusta, jossa tuosta 
asetuksesta on apua?

Harri