]> git.lyx.org Git - lyx.git/blobdiff - src/lyxserver.C
some tabular fixes for the problems reported by Helge
[lyx.git] / src / lyxserver.C
index 11f3d47aea8df785f1e216be66aa56d4a121d48f..a6213f9deefe2fce95e8559d0ff8c1d7d8daef65 100644 (file)
@@ -1,12 +1,15 @@
-/* This file is part of
- * ======================================================
- *
- *           LyX, The Document Processor
+/**
+ * \file lyxserver.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           Copyright 1995 Matthias Ettrich
- *           Copyright 1995-2001 The LyX Team.
+ * \author Lars Gullik Bjønnes
+ * \author Jean-Marc Lasgouttes
+ * \author Angus Leeming
+ * \author John Levon
  *
- * ====================================================== */
+ * Full author contact details are available in file CREDITS.
+ */
 
 /**
   Docu   : To use the lyxserver define the name of the pipe in your
 
 #include <config.h>
 
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <unistd.h>
-#include <fcntl.h>
-#include <cerrno>
-#include FORMS_H_LOCATION
-
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "lyxserver.h"
-#include "lyx_main.h"
 #include "debug.h"
+#include "funcrequest.h"
+#include "LyXAction.h"
 #include "lyxfunc.h"
 #include "support/lstrings.h"
 #include "support/lyxlib.h"
+#include "frontends/lyx_gui.h"
+
+#include <boost/bind.hpp>
+
+#include <cerrno>
+#include <sys/stat.h>
+#include <fcntl.h>
 
 #ifdef __EMX__
 #include <cstdlib>
 #include "support/os2_errortable.h"
 #endif
 
+using lyx::support::compare;
+using lyx::support::rtrim;
+using lyx::support::split;
+using lyx::support::unlink;
+
 using std::endl;
+using std::string;
+
 
 // provide an empty mkfifo() if we do not have one. This disables the
 // lyxserver.
@@ -75,23 +82,6 @@ int mkfifo(char const * __path, mode_t __mode) {
 #endif
 
 
-/* === variables ========================================================= */
-
-extern "C" {
-
-       // C wrapper
-       static
-       void C_LyXComm_callback(int fd, void *v)
-       {
-               LyXComm::callback(fd, v);
-       }
-
-}
-
-
-// LyXComm class
-
-// Open pipes
 void LyXComm::openConnection()
 {
        lyxerr[Debug::LYXSERVER] << "LyXComm: Opening connection" << endl;
@@ -115,7 +105,7 @@ void LyXComm::openConnection()
                return;
 
        if ((outfd = startPipe(outPipeName(), true)) == -1) {
-               endPipe(infd, inPipeName());
+               endPipe(infd, inPipeName(), false);
                return;
        }
 
@@ -148,16 +138,15 @@ void LyXComm::closeConnection()
                return;
        }
 
-       endPipe(infd, inPipeName());
-       endPipe(outfd, outPipeName());
+       endPipe(infd, inPipeName(), false);
+       endPipe(outfd, outPipeName(), true);
 
        ready = false;
 }
 
+
 int LyXComm::startPipe(string const & filename, bool write)
 {
-       int fd;
-
 #ifdef __EMX__
        HPIPE os2fd;
        APIRET rc;
@@ -190,7 +179,7 @@ int LyXComm::startPipe(string const & filename, bool write)
        };
        // Imported handles can be used both with OS/2 APIs and emx
        // library functions.
-       fd = _imphandle(os2fd);
+       int const fd = _imphandle(os2fd);
 #else
        if (::access(filename.c_str(), F_OK) == 0) {
                lyxerr << "LyXComm: Pipe " << filename << " already exists.\n"
@@ -205,28 +194,34 @@ int LyXComm::startPipe(string const & filename, bool write)
                       << strerror(errno) << endl;
                return -1;
        };
-       fd = ::open(filename.c_str(), write ? (O_RDWR) : (O_RDONLY|O_NONBLOCK));
+       int const fd = ::open(filename.c_str(),
+                             write ? (O_RDWR) : (O_RDONLY|O_NONBLOCK));
 #endif
 
        if (fd < 0) {
                lyxerr << "LyXComm: Could not open pipe " << filename << '\n'
                       << strerror(errno) << endl;
-               lyx::unlink(filename);
+               unlink(filename);
                return -1;
        }
 
-       if (!write)
-               fl_add_io_callback(fd, FL_READ, C_LyXComm_callback, this);
+       if (!write) {
+               lyx_gui::register_socket_callback(fd, boost::bind(&LyXComm::read_ready, this));
+       }
 
        return fd;
 }
 
 
-void LyXComm::endPipe(int & fd, string const & filename)
+void LyXComm::endPipe(int & fd, string const & filename, bool write)
 {
        if (fd < 0)
                return;
 
+       if (!write) {
+               lyx_gui::unregister_socket_callback(fd);
+       }
+
 #ifdef __EMX__
        APIRET rc;
        int errnum;
@@ -247,7 +242,7 @@ void LyXComm::endPipe(int & fd, string const & filename)
 
 // OS/2 pipes are deleted automatically
 #ifndef __EMX__
-       if (lyx::unlink(filename) < 0) {
+       if (unlink(filename) < 0) {
                lyxerr << "LyXComm: Could not remove pipe " << filename
                       << '\n' << strerror(errno) << endl;
        };
@@ -260,75 +255,65 @@ void LyXComm::endPipe(int & fd, string const & filename)
 void LyXComm::emergencyCleanup()
 {
        if (!pipename.empty()) {
-               endPipe(infd, inPipeName());
-               endPipe(outfd, outPipeName());
+               endPipe(infd, inPipeName(), false);
+               endPipe(outfd, outPipeName(), true);
        }
 }
 
 
 // Receives messages and sends then to client
-void LyXComm::callback(int fd, void *v)
+void LyXComm::read_ready()
 {
-       LyXComm * c = static_cast<LyXComm*>(v);
+       // nb! make read_buffer_ a class-member for multiple sessions
+       static string read_buffer_;
+       read_buffer_.erase();
 
-       if (lyxerr.debugging(Debug::LYXSERVER)) {
-               lyxerr << "LyXComm: Receiving from fd " << fd << endl;
-       }
-
-       const int CMDBUFLEN = 100;
-       char charbuf[CMDBUFLEN];
-       string cmd;
-// nb! make lsbuf a class-member for multiple sessions
-       static string lsbuf;
+       int const charbuf_size = 100;
+       char charbuf[charbuf_size];
 
        errno = 0;
        int status;
        // the single = is intended here.
-       while ((status = read(fd, charbuf, CMDBUFLEN-1)))
-       {// break and return in loop
-               if (status > 0) // got something
-               {
-                       charbuf[status]= '\0'; // turn it into a c string
-                       lsbuf += strip(charbuf, '\r');
+       while ((status = ::read(infd, charbuf, charbuf_size - 1))) {
+
+               if (status > 0) {
+                       charbuf[status] = '\0'; // turn it into a c string
+                       read_buffer_ += rtrim(charbuf, "\r");
                        // commit any commands read
-                       while (lsbuf.find('\n') != string::npos) // while still
-                                                       // commands
-                                                       // left
-                       {
+                       while (read_buffer_.find('\n') != string::npos) {
                                // split() grabs the entire string if
                                // the delim /wasn't/ found. ?:-P
-                               lsbuf= split(lsbuf, cmd,'\n');
+                               string cmd;
+                               read_buffer_= split(read_buffer_, cmd,'\n');
                                lyxerr[Debug::LYXSERVER]
                                        << "LyXComm: status:" << status
-                                       << ", lsbuf:" << lsbuf
+                                       << ", read_buffer_:" << read_buffer_
                                        << ", cmd:" << cmd << endl;
                                if (!cmd.empty())
-                                       c->clientcb(c->client, cmd);
+                                       clientcb(client, cmd);
                                        //\n or not \n?
                        }
                }
-               if (errno == EAGAIN)
-               {  // EAGAIN is not really an error , it means we're
-                  // only reading too fast for the writing process on
-                  // the other end of the pipe.
+               if (errno == EAGAIN) {
                        errno = 0;
-                       return; // up to libforms select-loop (*crunch*)
+                       return;
                }
-               if (errno != 0)
-               {
+               if (errno != 0) {
                        lyxerr << "LyXComm: " << strerror(errno) << endl;
-                       if (!lsbuf.empty())
-                       {
-                               lyxerr << "LyxComm: truncated command: "
-                                      << lsbuf << endl;
-                               lsbuf.erase();
+                       if (!read_buffer_.empty()) {
+                               lyxerr << "LyXComm: truncated command: "
+                                      << read_buffer_ << endl;
+                               read_buffer_.erase();
                        }
                        break; // reset connection
                }
        }
-       c->closeConnection();
-       c->openConnection();
-       errno= 0;
+
+       // The connection gets reset in errno != EAGAIN
+       // Why does it need to be reset if errno == 0?
+       closeConnection();
+       openConnection();
+       errno = 0;
 }
 
 
@@ -369,6 +354,18 @@ void LyXComm::send(string const & msg)
 }
 
 
+string const LyXComm::inPipeName() const
+{
+       return pipename + string(".in");
+}
+
+
+string const LyXComm::outPipeName() const
+{
+       return pipename + string(".out");
+}
+
+
 // LyXServer class
 
 LyXServer::~LyXServer()
@@ -408,7 +405,8 @@ void LyXServer::callback(LyXServer * serv, string const & msg)
                if (compare(p, "LYXSRV:", 7) == 0) {
                        server_only = true;
                } else if (0 != compare(p, "LYXCMD:", 7)) {
-                       lyxerr << "LyXServer: Unknown request \"" << p << "\"" << endl;
+                       lyxerr << "LyXServer: Unknown request \""
+                              << p << '"' << endl;
                        return;
                }
                p += 7;
@@ -484,7 +482,7 @@ void LyXServer::callback(LyXServer * serv, string const & msg)
                                }
                        } else {
                                lyxerr <<"LyXServer: Undefined server command "
-                                      << cmd << "." << endl;
+                                      << cmd << '.' << endl;
                        }
                        return;
                }
@@ -498,9 +496,9 @@ void LyXServer::callback(LyXServer * serv, string const & msg)
                        // support currently. (Lgb)
 
 
-                       serv->func->verboseDispatch(cmd + ' ' + arg, false);
+                       serv->func->dispatch(FuncRequest(lyxaction.lookupFunc(cmd), arg));
                        string const rval = serv->func->getMessage();
-                       
+
                        //modified june 1999 stefano@zool.su.se:
                        //all commands produce an INFO or ERROR message
                        //in the output pipe, even if they do not return
@@ -510,7 +508,7 @@ void LyXServer::callback(LyXServer * serv, string const & msg)
                                buf = "ERROR:";
                        else
                                buf = "INFO:";
-                       buf += client + ":" + cmd + ":" +  rval + "\n";
+                       buf += client + ':' + cmd + ':' +  rval + '\n';
                        serv->pipes.send(buf);
 
                        // !!! we don't do any error checking -
@@ -526,7 +524,7 @@ void LyXServer::callback(LyXServer * serv, string const & msg)
 
 
 /* ---F+------------------------------------------------------------------ *\
-   Function  : LyxNotifyClient
+   Function  : LyXNotifyClient
    Called by : WorkAreaKeyPress
    Purpose   : send a notify messge to a client
    Parameters: s - string to send