]> git.lyx.org Git - lyx.git/blobdiff - src/sp_spell.C
make a CURSOR update if we inserted a character (fix #312).
[lyx.git] / src / sp_spell.C
index e10adc3f5134e2d7cd4861df7af9d3f27207a461..a47903f65058f456c2b653bc1a6bfbceba6d720d 100644 (file)
@@ -1,5 +1,5 @@
 /* This file is part of
- * ====================================================== 
+ * ======================================================
  *
  *           LyX, The Document Processor
  *
 #pragma implementation
 #endif
 
-#ifdef USE_PSPELL
-
+#include <sys/types.h>
+#include <sys/wait.h>
 #include <unistd.h>
 #include <fcntl.h>
 #include <stdio.h>
 
+#if TIME_WITH_SYS_TIME
+# include <sys/time.h>
+# include <ctime>
+#else
+# if HAVE_SYS_TIME_H
+#  include <sys/time.h>
+# else
+#  include <ctime>
+# endif
+#endif
+
 #ifdef HAVE_SYS_SELECT_H
 # ifdef HAVE_STRINGS_H
    // <strings.h> is needed at least on AIX because FD_ZERO uses bzero().
 
 #include "LString.h"
 #include "support/lstrings.h"
+#include "lyxrc.h"
+#include "language.h"
+#include "debug.h"
+#include "encoding.h"
+#include "sp_ispell.h"
+
+using std::endl;
+
+namespace {
+       /// pid for the `ispell' process.
+       pid_t isp_pid = -1;
+}
+
+///
+// ------------------- start special pspell code/class --------------------
+///
+#ifdef USE_PSPELL
+
 #include "support/LAssert.h"
 
 #define USE_ORIGINAL_MANAGER_FUNCS 1
 
 #include "sp_pspell.h"
 
-extern void sigchldchecker(pid_t pid, int *status);
-
 
-PSpell::PSpell() 
+PSpell::PSpell()
+       : sc(0), els(0), spell_error_object(0), flag(ISP_UNKNOWN),
+         alive_(false)
 {
-       els = 0;
-       sc = 0;
-       spell_error_object = 0;
-       flag = ISP_UNKNOWN;
 }
 
+
 PSpell::PSpell(BufferParams const & params, string const & lang)
+       : sc(0), els(0), spell_error_object(0), flag(ISP_UNKNOWN),
+         alive_(false)
 {
-       els = 0;
-       sc = 0;
-       spell_error_object = 0;
-       flag = ISP_UNKNOWN;
        initialize(params, lang);
 }
 
-PSpell::~PSpell() 
+
+PSpell::~PSpell()
 {
        cleanUp();
        close();
        if (els)
                delete_pspell_string_emulation(els);
 }
-       
+
 
 void PSpell::initialize(BufferParams const &, string const & lang)
 {
        PspellConfig * config = new_pspell_config();
-       string code;
-       split(lang, code, '_');
-       config->replace("language-tag", code.c_str());
+       config->replace("language-tag", lang.c_str());
        spell_error_object = new_pspell_manager(config);
        if (pspell_error_number(spell_error_object) != 0) {
                error_ = pspell_error_message(spell_error_object);
@@ -85,11 +108,12 @@ void PSpell::initialize(BufferParams const &, string const & lang)
                error_ = 0;
                sc = to_pspell_manager(spell_error_object);
                spell_error_object = 0;
+               alive_ = true;
        }
 }
 
 
-void PSpell::cleanUp() 
+void PSpell::cleanUp()
 {
        if (spell_error_object) {
                delete_pspell_can_have_error(spell_error_object);
@@ -113,9 +137,9 @@ enum PSpell::spellStatus PSpell::check(string const & word)
                        pspell_manager_suggest(sc, word.c_str());
                lyx::Assert(sugs != 0);
                els = pspell_word_list_elements(sugs);
-               if (pspell_word_list_empty(sugs)) 
+               if (pspell_word_list_empty(sugs))
                        flag = ISP_UNKNOWN;
-               else 
+               else
                        flag = ISP_MISSED;
        }
        return flag;
@@ -136,7 +160,7 @@ void PSpell::insert(string const & word)
 }
 
 
-void PSpell::accept(string const & word) 
+void PSpell::accept(string const & word)
 {
        if (sc)
                pspell_manager_add_to_session(sc, word.c_str());
@@ -154,89 +178,30 @@ char const * PSpell::nextMiss()
 {
        if (els)
                return pspell_string_emulation_next(els);
+       return "";
 }
 
+
 char const * PSpell::error()
 {
        return error_;
 }
 
-void PSpell::sigchldhandler(pid_t pid, int * status)
-{
-       sigchldchecker(pid, status);
-}
-
-
-
-#else
-
-/*
- *This file is part of
- * ====================================================== 
- * 
- *           LyX, The Document Processor
- *      
- *         Copyright 1995 Matthias Ettrich
- *          Copyright 1995-1998 The LyX Team
- *
- * ====================================================== 
- */
-
-#include <unistd.h>
-#include <fcntl.h>
-#include <stdio.h>
 
-#if TIME_WITH_SYS_TIME
-# include <sys/time.h>
-# include <ctime>
-#else
-# if HAVE_SYS_TIME_H
-#  include <sys/time.h>
-# else
-#  include <ctime>
-# endif
 #endif
 
-#ifdef HAVE_SYS_SELECT_H
-# ifdef HAVE_STRINGS_H
-   // <strings.h> is needed at least on AIX because FD_ZERO uses bzero().
-   // BUT we cannot include both string.h and strings.h on Irix 6.5 :(
-#  ifdef _AIX
-#   include <strings.h>
-#  endif
-# endif
-#include <sys/select.h>
-#endif
-
-#include "LString.h"
-#include "support/lstrings.h"
-#include "lyxrc.h"
-#include "debug.h"
-#include "encoding.h"
-#include "sp_ispell.h"
-
-using std::endl;
-
-/// can be found in src/insets/figinset.C
-extern void sigchldchecker(pid_t pid, int * status);
-
-namespace {
-/// pid for the `ispell' process. 
-pid_t isp_pid = -1; 
-}
-
+///
+// ------------------- start special ispell code/class --------------------
+///
 
 ISpell::ISpell()
-{
-       str = 0;
-       flag = ISP_UNKNOWN;
-}
+       : str(0), flag(ISP_UNKNOWN)
+{}
 
 
 ISpell::ISpell(BufferParams const & params, string const & lang)
+       : str(0), flag(ISP_UNKNOWN)
 {
-       str = 0;
-       flag = ISP_UNKNOWN;
        initialize(params, lang);
 }
 
@@ -260,7 +225,8 @@ char const * ISpell::nextMiss()
 void ISpell::initialize(BufferParams const & params, string const & lang)
 {
        static char o_buf[BUFSIZ];  // jc: it could be smaller
-       int pipein[2], pipeout[2];
+       int pipein[2];
+       int pipeout[2];
        char * argv[14];
        int argc;
 
@@ -294,8 +260,8 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
                       << endl;
                goto END;
        }
-       
-       if (isp_pid == 0) {        
+
+       if (isp_pid == 0) {
                /* child process */
                dup2(pipein[0], STDIN_FILENO);
                dup2(pipeout[1], STDOUT_FILENO);
@@ -330,16 +296,16 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
                } else {
                        // Report run-together words with
                        // missing blanks as errors
-                       tmp = new char[3]; 
+                       tmp = new char[3];
                        string("-B").copy(tmp, 2); tmp[2] = '\0';
-                       argv[argc++] = tmp; 
+                       argv[argc++] = tmp;
                }
                if (lyxrc.isp_use_esc_chars) {
                        // Specify additional characters that
                        // can be part of a word
                        tmp = new char[3];
                        string("-w").copy(tmp, 2); tmp[2] = '\0';
-                       argv[argc++] = tmp; 
+                       argv[argc++] = tmp;
                        // Put the escape chars in ""s
                        string tms = "\"" + lyxrc.isp_esc_chars + "\"";
                        tmp = new char[tms.length() + 1];
@@ -364,7 +330,8 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
                                : params.inputenc;
                        string::size_type n = enc.length();
                        tmp = new char[3];
-                       string("-T").copy(tmp, 2); tmp[2] = '\0';
+                       string("-T").copy(tmp, 2);
+                       tmp[2] = '\0';
                        argv[argc++] = tmp; // Input encoding
                        tmp = new char[n + 1];
                        enc.copy(tmp, n);
@@ -378,9 +345,9 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
 
                // free the memory used by string::copy in the
                // setup of argv
-               for (int i= 0; i < argc -1; ++i)
+               for (int i = 0; i < argc - 1; ++i)
                        delete[] argv[i];
-               
+
                lyxerr << "LyX: Failed to start ispell!" << endl;
                _exit(0);
        }
@@ -401,17 +368,17 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
 
                // Configure provides us with macros which are supposed to do
                // the right typecast.
-               retval = select(SELECT_TYPE_ARG1 (pipeout[0]+1), 
-                               SELECT_TYPE_ARG234 (&infds), 
-                               0, 
-                               0, 
+               retval = select(SELECT_TYPE_ARG1 (pipeout[0]+1),
+                               SELECT_TYPE_ARG234 (&infds),
+                               0,
+                               0,
                                SELECT_TYPE_ARG5 (&tv));
 
                if (retval > 0) {
                        // Ok, do the reading. We don't have to FD_ISSET since
                        // there is only one fd in infds.
                        fgets(buf, 2048, in);
-               
+
                        // determine if the spell checker is really Aspell
                        if (strstr(buf, "Aspell"))
                                actual_spell_checker = ASC_ASPELL;
@@ -420,7 +387,7 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
 
                        fputs("!\n", out); // Set terse mode (silently accept correct words)
 
-               
+
                } else if (retval == 0) {
                        // timeout. Give nice message to user.
                        lyxerr << "Ispell read timed out, what now?" << endl;
@@ -438,26 +405,40 @@ void ISpell::initialize(BufferParams const & params, string const & lang)
        }
   END:
        if (isp_pid == -1) {
-               error_ = 
+               error_ =
                        "\n\n"
-                       "The ispell-process has died for some reason. *One* possible reason\n"
-                       "could be that you do not have a dictionary file\n"
-                       "for the language of this document installed.\n"
-                       "Check /usr/lib/ispell or set another\n"
-                       "dictionary in the Spellchecker Options menu.";
+                       "The spellcheck-process has died for some reason.\n"
+                       "*One* possible reason could be that you do not have\n"
+                       "a dictionary file for the language of this document\n"
+                       "installed.\n"
+                       "Check your spellchecker or set another dictionary\n"
+                       "in the Spellchecker Options menu.\n\n";
        } else {
                error_ = 0;
        }
 }
 
 
+/* FIXME: this is a minimalist solution until the above
+ * code is able to work with forkedcall.h. We only need
+ * to reap the zombies here.
+ */
+void reapSpellchecker(void)
+{
+       if (isp_pid == -1)
+               return;
+
+       waitpid(isp_pid, 0, WNOHANG);
+}
+
+
 bool ISpell::alive()
 {
        return isp_pid != -1;
 }
 
 
-void ISpell::cleanUp() 
+void ISpell::cleanUp()
 {
        ::fclose(out);
 }
@@ -469,12 +450,12 @@ enum ISpell::spellStatus ISpell::check(string const & word)
 
        ::fputs(word.c_str(), out);
        ::fputc('\n', out);
-  
+
        char buf[1024];
-       ::fgets(buf, 1024, in); 
-  
-       /* I think we have to check if ispell is still alive here because
-          the signal-handler could have disabled blocking on the fd */
+       ::fgets(buf, 1024, in);
+
+       // I think we have to check if ispell is still alive here because
+       // the signal-handler could have disabled blocking on the fd
        if (!alive()) return ISP_UNKNOWN;
 
        switch (*buf) {
@@ -517,10 +498,10 @@ enum ISpell::spellStatus ISpell::check(string const & word)
 
 void ISpell::close()
 {
-        // Note: If you decide to optimize this out when it is not 
-        // needed please note that when Aspell is used this command 
-        // is also needed to save the replacement dictionary.
-        // -- Kevin Atkinson (kevinatk@home.com)
+       // Note: If you decide to optimize this out when it is not
+       // needed please note that when Aspell is used this command
+       // is also needed to save the replacement dictionary.
+       // -- Kevin Atkinson (kevinatk@home.com)
 
        fputs("#\n", out); // Save personal dictionary
 
@@ -537,7 +518,7 @@ void ISpell::insert(string const & word)
 }
 
 
-void ISpell::accept(string const & word) 
+void ISpell::accept(string const & word)
 {
        ::fputc('@', out); // Accept in this session
        ::fputs(word.c_str(), out);
@@ -547,34 +528,17 @@ void ISpell::accept(string const & word)
 
 void ISpell::store(string const & mis, string const & cor)
 {
-        if (actual_spell_checker == ASC_ASPELL) {
-                ::fputs("$$ra ", out);
-                ::fputs(mis.c_str(), out);
-                ::fputc(',', out);
-                ::fputs(cor.c_str(), out);
-                ::fputc('\n', out);
-        }
-}
-
-
-void ISpell::sigchldhandler(pid_t pid, int * status)
-{
-       if (isp_pid > 0) {
-               if (pid == isp_pid) {
-                       isp_pid= -1;
-                       fcntl(isp_fd, F_SETFL, O_NONBLOCK); /* set the file descriptor
-                                                              to nonblocking so we can 
-                                                              continue */
-               }
+       if (actual_spell_checker == ASC_ASPELL) {
+               ::fputs("$$ra ", out);
+               ::fputs(mis.c_str(), out);
+               ::fputc(',', out);
+               ::fputs(cor.c_str(), out);
+               ::fputc('\n', out);
        }
-       sigchldchecker(pid, status);
 }
 
+
 char const * ISpell::error()
 {
        return error_;
 }
-
-
-
-#endif