On 15-Jul-2001 Edwin Leuven wrote:

> works in the attached patch.

-                       view().partialUpdate(2);
-                       hide();
+                       clearParams();

and then:

-void ControlSpellchecker::quit()
+void ControlSpellchecker::clearParams()
 {
+       if (!speller_) return;
+       
        if (speller_->alive()) {
                speller_->close();
...

This doesn't the right stuff when compiling with pspell. My PSpell installation
has some problems to open the right aspell-module, so it will put an error
message int error_ which in this version is never outputed, moreover you tell
the user that 0 words have been spellchecked but you don't tell'em the error.

Please fix this before I submit the patch. You see that I removed the call
to quit for that reson from the if (error_) part and did output the error
message which caused the abord instead and then deleted the speller_. This
seems more then reasonable to me, what do you think is not good there?

> About pspell, it works on my system: pspell 0.12.2. Perhaps older versions of 
> pspell have some libtool problems. 

Well I would be interested how it works on your machine, here it does still
not work!

         Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._
Dr. Jürgen Vigna        E-Mail:  [EMAIL PROTECTED]
Italienallee 13/N       Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen           Web:     http://www.sad.it/~jug
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Weinberg's Principle:
        An expert is a person who avoids the small errors while
        sweeping on to the grand fallacy.

Reply via email to