On Tuesday 23 July 2002 12:35, Lars Gullik Bjønnes wrote:
>
> Ok... can you try to describe how you plan to do all this, so that we
> can have a small discussion before you begin?

  Ok, in a different thread and after the lunch. :-)

> >> _how_ can that be...
> |
> |   The time or the fact that it compiled straight away?
>
> That it compiled straight away...

Ok, I have a problem with 1.0.1 and 1.0.2 at least because of this patch:

--- ../lyx-1.0.0/src/spellchecker.C     Wed Jan  6 06:51:51 1999
+++ src/spellchecker.C  Fri Feb  5 17:28:32 1999
@@ -339,7 +339,13 @@
                        // but it can't really hurt.
        tv.tv_usec = 0;

-       retval = select(pipeout[0]+1, &infds, 0, 0, &tv);
+       // 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,
+                       SELECT_TYPE_ARG5 (&tv));

        if (retval > 0) {
                // Ok, do the reading. We don't have to FD_ISSET since

  I get an error with 1.0.1:
g++ -c -g -O2 -I. -I. -I../images   -I/usr/X11R6/include spellchecker.C
spellchecker.C: In function `void create_ispell_pipe (const LString &)':
spellchecker.C:348: cannot convert `int *' to `fd_set *' for argument
`2' to `select (int, fd_set *, fd_set *, fd_set *, timeval *)'

  Should I revert this patch? Why does it fails?
  I was expecting it to be safer with those macros, but here g++ (2.96-110) 
doesn't complain with 1.0.0.

-- 
José Abílio

Reply via email to