Re: lynx-dev lynx2.8.5dev.16i.patch.gz

[Leonid Pauzner]

4-Jan-2004 20:49 Thomas Dickey wrote:
> updated patch

> ftp://invisible-island.net/temp/lynx2.8.5dev.16i.patch.gz


A quick reading show me a few chunks of TRSTable.c thich could be reverted,
I mean 'struct STable_cellinfo' and 'struct STable_rowinfo' definitions,
to use short instead of int's.


Index: src/TRSTable.c
--- lynx2.8.5dev.16+/src/TRSTable.c     2003-06-01 21:32:28.000000000 -0400
+++ lynx2.8.5dev.16h/src/TRSTable.c     2003-12-30 21:14:21.000000000 -0500
@@ -79,10 +71,10 @@
                                   contentless cells (and cells we do
                                   not want to measure and count?),
                                   line-of-the-start otherwise.  */
-       short   pos;            /* column where cell starts */
-       short   len;            /* number of character positions */
-       short   colspan;        /* number of columns to span */
-       short   alignment;      /* one of HT_LEFT, HT_CENTER, HT_RIGHT,
+       int     pos;            /* column where cell starts */
+       int     len;            /* number of character positions */
+       int     colspan;        /* number of columns to span */
+       int     alignment;      /* one of HT_LEFT, HT_CENTER, HT_RIGHT,
                                   or RESERVEDCELL */
 } STable_cellinfo;

@@ -98,13 +90,12 @@
 #define OFFSET_IS_VALID                        8
 #define OFFSET_IS_VALID_LAST_CELL      0x10
 #define BELIEVE_OFFSET                 0x20
-#define IS_CONTINUATION_OF_MULTICELL   0x40

 typedef struct _STable_rowinfo {
     /* Each row may be displayed on many display lines, but we fix up
        positions of cells on this display line only: */
        int     Line;           /* lineno in doc (zero-based) */
-       short   ncells;         /* number of table cells */
+       int     ncells;         /* number of table cells */

     /* What is the meaning of this?!  It is set if:
        [search for     def of fixed_line       below]
@@ -128,25 +119,15 @@
        REMARK: If this variable is not set, but icell_core is, Line is
        reset to the line of icell_core.
      */
-       short   fixed_line;     /* if we have a 'core' line of cells */
+       BOOL    fixed_line;     /* if we have a 'core' line of cells */
        enum ended_state ended; /* if we saw </tr> etc */
-       short   content;        /* Whether contains end-of-cell etc */
-       short   offset;         /* >=0 after line break in a multiline cell */
-       short   allocated;      /* number of table cells allocated or 0
-                                  if the .cells should not be free()ed */
-       short   alignment;      /* global align attribute for this row */
+       int     content;        /* Whether contains end-of-cell etc */
+       int     offset;         /* >=0 after line break in a multiline cell */
+       int     allocated;      /* number of table cells allocated */
        STable_cellinfo * cells;
+       int     alignment;      /* global align attribute for this row */
 } STable_rowinfo;



; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to address@hidden