]> git.lyx.org Git - lyx.git/blobdiff - src/tabular.C
more code in the menu backend == less code in the menu frontends; add support for...
[lyx.git] / src / tabular.C
index d151ce58fc2155fdcbab65786f6e0bcbdb1388be..00dbed5f197745582287b880a90cb3f13eab7901 100644 (file)
@@ -1,43 +1,42 @@
 /* This file is part of
- * ====================================================== 
- * 
+ * ======================================================
+ *
  *           LyX, The Document Processor
- *      
- *           Copyright 2000-2001 The LyX Team.
+ *
+ *           Copyright 2000-2002 The LyX Team.
  *
  *           @author: Jürgen Vigna
  *
- * ====================================================== 
+ * ======================================================
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 // temporary until verified (08/08/2001 Jug)
 #define SPECIAL_COLUM_HANDLING 1
 
-#include <algorithm>
-#include <cstdlib>
-
 #include "tabular.h"
 #include "debug.h"
 #include "vspace.h"
 #include "layout.h"
-#include "frontends/Alert.h"
 #include "buffer.h"
 #include "BufferView.h"
-#include "Painter.h"
+#include "frontends/Painter.h"
 #include "LaTeXFeatures.h"
+#include "insets/insettabular.h"
+#include "insets/insettext.h"
 #include "support/lstrings.h"
 #include "support/lyxmanip.h"
 #include "support/LAssert.h"
-#include "insets/insettabular.h"
-#include "insets/insettext.h"
+#include "frontends/Alert.h"
 #include "gettext.h"
+#include "tabular_funcs.h"
+#include "lyxlex.h"
 
+#include <algorithm>
+#include <cstdlib>
+
+using std::abs;
 using std::ostream;
 using std::istream;
 using std::getline;
@@ -57,7 +56,8 @@ namespace {
 
 /// Define a few methods for the inner structs
 
-LyXTabular::cellstruct::cellstruct() 
+LyXTabular::cellstruct::cellstruct(BufferParams const & bg)
+       : inset(bg)
 {
        cellno = 0;
        width_of_cell = 0;
@@ -73,7 +73,7 @@ LyXTabular::cellstruct::cellstruct()
 }
 
 
-LyXTabular::rowstruct::rowstruct() 
+LyXTabular::rowstruct::rowstruct()
 {
        top_line = true;
        bottom_line = false;
@@ -87,7 +87,7 @@ LyXTabular::rowstruct::rowstruct()
 }
 
 
-LyXTabular::columnstruct::columnstruct() 
+LyXTabular::columnstruct::columnstruct()
 {
        left_line = true;
        right_line = false;
@@ -97,7 +97,7 @@ LyXTabular::columnstruct::columnstruct()
 }
 
 
-LyXTabular::lttype::lttype()
+LyXTabular::ltType::ltType()
 {
        topDL = false;
        bottomDL = false;
@@ -106,20 +106,22 @@ LyXTabular::lttype::lttype()
 
 
 /* konstruktor */
-LyXTabular::LyXTabular(InsetTabular * inset, int rows_arg, int columns_arg)
+LyXTabular::LyXTabular(BufferParams const & bp,
+                      InsetTabular * inset, int rows_arg, int columns_arg)
 {
        owner_ = inset;
        cur_cell = -1;
-       Init(rows_arg, columns_arg);
+       Init(bp, rows_arg, columns_arg);
 }
 
 
-LyXTabular::LyXTabular(InsetTabular * inset, LyXTabular const & lt,
-                       bool same_id)
+LyXTabular::LyXTabular(BufferParams const & bp,
+                      InsetTabular * inset, LyXTabular const & lt,
+                      bool same_id)
 {
        owner_ = inset;
        cur_cell = -1;
-       Init(lt.rows_, lt.columns_, &lt);
+       Init(bp, lt.rows_, lt.columns_, &lt);
        // we really should change again to have InsetText as a pointer
        // and allocate it then we would not have to do this stuff all
        // double!
@@ -151,7 +153,7 @@ LyXTabular::LyXTabular(Buffer const * buf, InsetTabular * inset, LyXLex & lex)
 LyXTabular & LyXTabular::operator=(LyXTabular const & lt)
 {
 #if 0
-#warning This while method should look like this: (Lgb)
+#warning This whole method should look like this: (Lgb)
 
                LyXTabular tmp(lt);
                tmp.swap(*this);
@@ -173,9 +175,10 @@ LyXTabular & LyXTabular::operator=(LyXTabular const & lt)
 }
 
 
-LyXTabular * LyXTabular::clone(InsetTabular * inset, bool same_id)
+LyXTabular * LyXTabular::clone(BufferParams const & bp,
+                              InsetTabular * inset, bool same_id)
 {
-       LyXTabular * result = new LyXTabular(inset, *this, same_id);
+       LyXTabular * result = new LyXTabular(bp, inset, *this, same_id);
 #if 0
        // don't know if this is good but I need to Clone also
        // the text-insets here, this is for the Undo-facility!
@@ -190,14 +193,15 @@ LyXTabular * LyXTabular::clone(InsetTabular * inset, bool same_id)
 }
 
 
-/* activates all lines and sets all widths to 0 */ 
-void LyXTabular::Init(int rows_arg, int columns_arg, LyXTabular const * lt)
+/* activates all lines and sets all widths to 0 */
+void LyXTabular::Init(BufferParams const & bp,
+                     int rows_arg, int columns_arg, LyXTabular const * lt)
 {
        rows_ = rows_arg;
        columns_ = columns_arg;
        row_info = row_vector(rows_, rowstruct());
        column_info = column_vector(columns_, columnstruct());
-       cell_info = cell_vvector(rows_, cell_vector(columns_, cellstruct()));
+       cell_info = cell_vvector(rows_, cell_vector(columns_, cellstruct(bp)));
 
        if (lt) {
                operator=(*lt);
@@ -220,7 +224,7 @@ void LyXTabular::Init(int rows_arg, int columns_arg, LyXTabular const * lt)
                calculate_width_of_column(i);
        }
        column_info.back().right_line = true;
-   
+
        calculate_width_of_tabular();
 
        rowofcell = vector<int>();
@@ -231,23 +235,23 @@ void LyXTabular::Init(int rows_arg, int columns_arg, LyXTabular const * lt)
 }
 
 
-void LyXTabular::AppendRow(int cell)
+void LyXTabular::AppendRow(BufferParams const & bp, int cell)
 {
        ++rows_;
-   
+
        int row = row_of_cell(cell);
 
        row_vector::iterator rit = row_info.begin() + row;
        row_info.insert(rit, rowstruct());
        // now set the values of the row before
-       row_info[row] = row_info[row+1];
+       row_info[row] = row_info[row + 1];
 
 #if 0
        cell_vvector::iterator cit = cell_info.begin() + row;
-       cell_info.insert(cit, vector<cellstruct>(columns_, cellstruct()));
+       cell_info.insert(cit, vector<cellstruct>(columns_, cellstruct(bp)));
 #else
        cell_vvector c_info = cell_vvector(rows_, cell_vector(columns_,
-                                                                                                                 cellstruct()));
+                                                             cellstruct(bp)));
 
        for (int i = 0; i <= row; ++i) {
                for (int j = 0; j < columns_; ++j) {
@@ -262,7 +266,9 @@ void LyXTabular::AppendRow(int cell)
        cell_info = c_info;
        ++row;
        for (int j = 0; j < columns_; ++j) {
-               cell_info[row][j].inset.clear();
+               cell_info[row][j].inset.clear(false);
+               if (bp.tracking_changes)
+                       cell_info[row][j].inset.markNew(true);
        }
 #endif
        Reinit();
@@ -272,7 +278,7 @@ void LyXTabular::AppendRow(int cell)
 void LyXTabular::DeleteRow(int row)
 {
        if (rows_ == 1) return; // Not allowed to delete last row
-       
+
        row_info.erase(row_info.begin() + row); //&row_info[row]);
        cell_info.erase(cell_info.begin() + row); //&cell_info[row]);
        --rows_;
@@ -280,17 +286,17 @@ void LyXTabular::DeleteRow(int row)
 }
 
 
-void LyXTabular::AppendColumn(int cell)
+void LyXTabular::AppendColumn(BufferParams const & bp, int cell)
 {
        ++columns_;
-   
+
        cell_vvector c_info = cell_vvector(rows_, cell_vector(columns_,
-                                                                                                                 cellstruct()));
+                                                             cellstruct(bp)));
        int const column = column_of_cell(cell);
        column_vector::iterator cit = column_info.begin() + column + 1;
        column_info.insert(cit, columnstruct());
        // set the column values of the column before
-       column_info[column+1] = column_info[column];
+       column_info[column + 1] = column_info[column];
 
        for (int i = 0; i < rows_; ++i) {
                for (int j = 0; j <= column; ++j) {
@@ -300,19 +306,22 @@ void LyXTabular::AppendColumn(int cell)
                        c_info[i][j] = cell_info[i][j - 1];
                }
                // care about multicolumns
-               if (c_info[i][column + 1].multicolumn==CELL_BEGIN_OF_MULTICOLUMN) {
+               if (c_info[i][column + 1].multicolumn==CELL_BEGIN_OF_MULTICOLUMN)
+               {
                        c_info[i][column + 1].multicolumn = CELL_PART_OF_MULTICOLUMN;
                }
-               if ((column + 1) == columns_ ||
-                       c_info[i][column + 2].multicolumn != CELL_PART_OF_MULTICOLUMN) {
+               if ((column + 2) >= columns_ ||
+                       c_info[i][column + 2].multicolumn != CELL_PART_OF_MULTICOLUMN)
+               {
                        c_info[i][column + 1].multicolumn = LyXTabular::CELL_NORMAL;
                }
        }
        cell_info = c_info;
        //++column;
        for (int i = 0; i < rows_; ++i) {
-               //cell_info[i][column].inset.clear();
-               cell_info[i][column + 1].inset.clear();
+               cell_info[i][column + 1].inset.clear(false);
+               if (bp.tracking_changes)
+                       cell_info[i][column + 1].inset.markNew(true);
        }
        Reinit();
 }
@@ -324,7 +333,7 @@ void LyXTabular::DeleteColumn(int column)
        //if (!(columns_ - 1))
        //return;
        if (columns_ == 1) return; // Not allowed to delete last column
-        
+
        column_info.erase(column_info.begin() + column);
        for (int i = 0; i < rows_; ++i) {
                cell_info[i].erase(cell_info[i].begin() + column);
@@ -350,7 +359,7 @@ void LyXTabular::Reinit(bool reset_widths)
                        }
                }
        }
-  
+
        for (int i = 0; i < columns_; ++i) {
                calculate_width_of_column(i);
        }
@@ -429,11 +438,11 @@ int LyXTabular::NumberOfCellsInRow(int cell) const
 }
 
 
-/* returns 1 if there is a topline, returns 0 if not */ 
+/* returns 1 if there is a topline, returns 0 if not */
 bool LyXTabular::TopLine(int cell, bool onlycolumn) const
 {
        int const row = row_of_cell(cell);
-       
+
        if (!onlycolumn && IsMultiColumn(cell))
                return cellinfo_of_cell(cell)->top_line;
        return row_info[row].top_line;
@@ -456,11 +465,13 @@ bool LyXTabular::BottomLine(int cell, bool onlycolumn) const
 
 bool LyXTabular::LeftLine(int cell, bool onlycolumn) const
 {
-       if (!onlycolumn && IsMultiColumn(cell)) {
+       if (!onlycolumn && IsMultiColumn(cell) &&
+               (IsFirstCellInRow(cell) || IsMultiColumn(cell-1)))
+       {
 #ifdef SPECIAL_COLUM_HANDLING
                if (cellinfo_of_cell(cell)->align_special.empty())
                        return cellinfo_of_cell(cell)->left_line;
-               return prefixIs(frontStrip(cellinfo_of_cell(cell)->align_special), "|");
+               return prefixIs(ltrim(cellinfo_of_cell(cell)->align_special), "|");
 #else
                return cellinfo_of_cell(cell)->left_line;
 #endif
@@ -468,7 +479,7 @@ bool LyXTabular::LeftLine(int cell, bool onlycolumn) const
 #ifdef SPECIAL_COLUM_HANDLING
        if (column_info[column_of_cell(cell)].align_special.empty())
                return column_info[column_of_cell(cell)].left_line;
-       return prefixIs(frontStrip(column_info[column_of_cell(cell)].align_special), "|");
+       return prefixIs(ltrim(column_info[column_of_cell(cell)].align_special), "|");
 #else
        return column_info[column_of_cell(cell)].left_line;
 #endif
@@ -477,11 +488,13 @@ bool LyXTabular::LeftLine(int cell, bool onlycolumn) const
 
 bool LyXTabular::RightLine(int cell, bool onlycolumn) const
 {
-       if (!onlycolumn && IsMultiColumn(cell)) {
+       if (!onlycolumn && IsMultiColumn(cell) &&
+               (IsLastCellInRow(cell) || IsMultiColumn(cell+1)))
+       {
 #ifdef SPECIAL_COLUM_HANDLING
                if (cellinfo_of_cell(cell)->align_special.empty())
                        return cellinfo_of_cell(cell)->right_line;
-               return suffixIs(strip(cellinfo_of_cell(cell)->align_special), "|");
+               return suffixIs(rtrim(cellinfo_of_cell(cell)->align_special), "|");
 #else
                return cellinfo_of_cell(cell)->right_line;
 #endif
@@ -489,14 +502,14 @@ bool LyXTabular::RightLine(int cell, bool onlycolumn) const
 #ifdef SPECIAL_COLUM_HANDLING
        if (column_info[column_of_cell(cell)].align_special.empty())
                return column_info[right_column_of_cell(cell)].right_line;
-       return suffixIs(strip(column_info[column_of_cell(cell)].align_special), "|");
+       return suffixIs(rtrim(column_info[column_of_cell(cell)].align_special), "|");
 #else
        return column_info[right_column_of_cell(cell)].right_line;
 #endif
 }
 
 
-bool LyXTabular::TopAlreadyDrawed(int cell) const
+bool LyXTabular::topAlreadyDrawn(int cell) const
 {
        int row = row_of_cell(cell);
        if ((row > 0) && !GetAdditionalHeight(row)) {
@@ -515,7 +528,7 @@ bool LyXTabular::TopAlreadyDrawed(int cell) const
 }
 
 
-bool LyXTabular::LeftAlreadyDrawed(int cell) const
+bool LyXTabular::leftAlreadyDrawn(int cell) const
 {
        int column = column_of_cell(cell);
        if (column > 0) {
@@ -526,7 +539,7 @@ bool LyXTabular::LeftAlreadyDrawed(int cell) const
                if (GetAdditionalWidth(cell_info[row][column].cellno))
                        return false;
 #ifdef SPECIAL_COLUM_HANDLING
-               return column_info[column].right_line;
+               return RightLine(cell_info[row][column].cellno);
 #else
                return RightLine(cell_info[row][column].cellno, true);
 #endif
@@ -549,7 +562,7 @@ int LyXTabular::GetAdditionalHeight(int row) const
        bool top = true;
        bool bottom = true;
 
-       for (int column = 0; column < columns_ - 1 && bottom; ++column) {
+       for (int column = 0; column < columns_ && bottom; ++column) {
                switch (cell_info[row - 1][column].multicolumn) {
                case LyXTabular::CELL_BEGIN_OF_MULTICOLUMN:
                        bottom = cell_info[row - 1][column].bottom_line;
@@ -558,8 +571,8 @@ int LyXTabular::GetAdditionalHeight(int row) const
                        bottom = row_info[row - 1].bottom_line;
                }
        }
-       for (int column = 0; column < columns_ - 1 && top; ++column) {
-               switch (cell_info[row][column].multicolumn){
+       for (int column = 0; column < columns_ && top; ++column) {
+               switch (cell_info[row][column].multicolumn) {
                case LyXTabular::CELL_BEGIN_OF_MULTICOLUMN:
                        top = cell_info[row][column].top_line;
                        break;
@@ -579,8 +592,8 @@ int LyXTabular::GetAdditionalWidth(int cell) const
        // used to get it back in text.C
        int const col = right_column_of_cell(cell);
        int const row = row_of_cell(cell);
-       if (col < columns_ - 1 && RightLine(cell, true) &&
-               LeftLine(cell_info[row][col+1].cellno, true)) // column_info[col+1].left_line)
+       if (col < columns_ - 1 && RightLine(cell) &&
+               LeftLine(cell_info[row][col+1].cellno)) // column_info[col+1].left_line)
        {
                return WIDTH_OF_LINE;
        } else {
@@ -589,7 +602,7 @@ int LyXTabular::GetAdditionalWidth(int cell) const
 }
 
 
-// returns the maximum over all rows 
+// returns the maximum over all rows
 int LyXTabular::GetWidthOfColumn(int cell) const
 {
        int const column1 = column_of_cell(cell);
@@ -608,15 +621,16 @@ int LyXTabular::GetWidthOfTabular() const
 }
 
 
-/* returns 1 if a complete update is necessary, otherwise 0 */ 
+/* returns 1 if a complete update is necessary, otherwise 0 */
 bool LyXTabular::SetWidthOfMulticolCell(int cell, int new_width)
 {
        if (!IsMultiColumn(cell))
                return false;
-       
+
        int const row = row_of_cell(cell);
        int const column1 = column_of_cell(cell);
        int const column2 = right_column_of_cell(cell);
+       int const old_val = cell_info[row][column2].width_of_cell;
 
        // first set columns to 0 so we can calculate the right width
        for (int i = column1; i <= column2; ++i) {
@@ -624,7 +638,6 @@ bool LyXTabular::SetWidthOfMulticolCell(int cell, int new_width)
        }
        // set the width to MAX_WIDTH until width > 0
        int width = (new_width + 2 * WIDTH_OF_LINE);
-
        int i = column1;
        for (; i < column2 && width > column_info[i].width_of_column; ++i) {
                cell_info[row][i].width_of_cell = column_info[i].width_of_column;
@@ -633,29 +646,42 @@ bool LyXTabular::SetWidthOfMulticolCell(int cell, int new_width)
        if (width > 0) {
                cell_info[row][i].width_of_cell = width;
        }
+       if (old_val != cell_info[row][column2].width_of_cell) {
+               // in this case we have to recalculate all multicolumn cells which
+               // have this column as one of theirs but not as last one
+               calculate_width_of_column_NMC(i);
+               recalculateMulticolumnsOfColumn(i);
+               calculate_width_of_column(i);
+       }
        return true;
 }
 
 
-void LyXTabular::recalculateMulticolCells(int cell, int new_width)
+void LyXTabular::recalculateMulticolumnsOfColumn(int column)
 {
-       int const row = row_of_cell(cell);
-       int const column1 = column_of_cell(cell);
-       int const column2 = right_column_of_cell(cell);
-
-       // first set columns to 0 so we can calculate the right width
-       int i = column1;
-       for (; i <= column2; ++i)
-               cell_info[row][i].width_of_cell = 0;
-       for (i = cell + 1; (i < numberofcells) && (!IsMultiColumn(i)); ++i)
-               ;
-       if (i < numberofcells)
-               recalculateMulticolCells(i, GetWidthOfCell(i) - (2 * WIDTH_OF_LINE));
-       SetWidthOfMulticolCell(cell, new_width);
+       // the last column does not have to be recalculated because all
+       // multicolumns will have here there last multicolumn cell which
+       // always will have the whole rest of the width of the cell.
+       if (column > (columns_ - 2))
+               return;
+       for(int row = 0; row < rows_; ++row) {
+               int mc = cell_info[row][column].multicolumn;
+               int nmc = cell_info[row][column+1].multicolumn;
+               // we only have to update multicolumns which do not have this
+               // column as their last column!
+               if (mc == CELL_BEGIN_OF_MULTICOLUMN ||
+                       ((mc == CELL_PART_OF_MULTICOLUMN) &&
+                        (nmc == CELL_PART_OF_MULTICOLUMN)))
+               {
+                       int const cellno = cell_info[row][column].cellno;
+                       SetWidthOfMulticolCell(cellno,
+                                              GetWidthOfCell(cellno)-(2 * WIDTH_OF_LINE));
+               }
+       }
 }
 
 
-/* returns 1 if a complete update is necessary, otherwise 0 */ 
+/* returns 1 if a complete update is necessary, otherwise 0 */
 bool LyXTabular::SetWidthOfCell(int cell, int new_width)
 {
        int const row = row_of_cell(cell);
@@ -685,16 +711,11 @@ bool LyXTabular::SetWidthOfCell(int cell, int new_width)
                width = (new_width + 2*WIDTH_OF_LINE + add_width);
                cell_info[row][column1].width_of_cell = width;
                tmp = calculate_width_of_column_NMC(column1);
+               if (tmp)
+                       recalculateMulticolumnsOfColumn(column1);
        }
        if (tmp) {
-               int i = 0;
-               for (; i<columns_; ++i)
-                       calculate_width_of_column_NMC(i);
-               for (i = 0; (i < numberofcells) && !IsMultiColumn(i); ++i)
-                       ;
-               if (i < numberofcells)
-                       recalculateMulticolCells(i, GetWidthOfCell(i)-(2 * WIDTH_OF_LINE));
-               for (i = 0; i < columns_; ++i)
+               for (int i = 0; i < columns_; ++i)
                        calculate_width_of_column(i);
                calculate_width_of_tabular();
                return true;
@@ -729,8 +750,9 @@ bool LyXTabular::SetColumnPWidth(int cell, LyXLength const & width)
        int const j = column_of_cell(cell);
 
        column_info[j].p_width = width;
-       if (flag) // do this only if there is a width
-               SetAlignment(cell, LYX_ALIGN_LEFT);
+       // This should not ne necessary anymore
+       //      if (flag) // do this only if there is a width
+       //      SetAlignment(cell, LYX_ALIGN_LEFT);
        for (int i = 0; i < rows_; ++i) {
                int c = GetCellNumber(i, j);
                flag = !GetPWidth(c).zero(); // because of multicolumns!
@@ -754,7 +776,7 @@ bool LyXTabular::SetMColumnPWidth(int cell, LyXLength const & width)
 
 
 bool LyXTabular::SetAlignSpecial(int cell, string const & special,
-                                 LyXTabular::Feature what)
+                                LyXTabular::Feature what)
 {
        if (what == SET_SPECIAL_MULTI)
                cellinfo_of_cell(cell)->align_special = special;
@@ -881,8 +903,8 @@ int LyXTabular::GetWidthOfCell(int cell) const
 int LyXTabular::GetBeginningOfTextInCell(int cell) const
 {
        int x = 0;
-   
-       switch (GetAlignment(cell)){
+
+       switch (GetAlignment(cell)) {
        case LYX_ALIGN_CENTER:
                x += (GetWidthOfColumn(cell) - GetWidthOfCell(cell)) / 2;
                break;
@@ -890,10 +912,10 @@ int LyXTabular::GetBeginningOfTextInCell(int cell) const
                x += GetWidthOfColumn(cell) - GetWidthOfCell(cell);
                // + GetAdditionalWidth(cell);
                break;
-       default: /* LYX_ALIGN_LEFT: nothing :-) */ 
+       default: /* LYX_ALIGN_LEFT: nothing :-) */
                break;
        }
-       
+
        // the LaTeX Way :-(
        x += WIDTH_OF_LINE;
        return x;
@@ -931,7 +953,7 @@ bool LyXTabular::calculate_width_of_column(int column)
 {
        int const old_column_width = column_info[column].width_of_column;
        int maximum = 0;
-       
+
        for (int i = 0; i < rows_; ++i) {
                maximum = max(cell_info[i][column].width_of_cell, maximum);
        }
@@ -941,16 +963,21 @@ bool LyXTabular::calculate_width_of_column(int column)
 
 
 //
-// calculate the with of the column without regarding REAL MultiColumn
-// cells. This means MultiColumn-cells spanning more than 1 column.
+// Calculate the columns regarding ONLY the normal cells and if this
+// column is inside a multicolumn cell then use it only if its the last
+// column of this multicolumn cell as this gives an added with to the
+// column, all the rest should be adapted!
 //
 bool LyXTabular::calculate_width_of_column_NMC(int column)
 {
        int const old_column_width = column_info[column].width_of_column;
        int max = 0;
        for (int i = 0; i < rows_; ++i) {
-               if (!IsMultiColumn(GetCellNumber(i, column), true) &&
-                       (cell_info[i][column].width_of_cell > max)) {
+               int cell = GetCellNumber(i, column);
+               bool ismulti = IsMultiColumn(cell, true);
+               if ((!ismulti || (column == right_column_of_cell(cell))) &&
+                       (cell_info[i][column].width_of_cell > max))
+               {
                        max = cell_info[i][column].width_of_cell;
                }
        }
@@ -999,114 +1026,6 @@ int LyXTabular::right_column_of_cell(int cell) const
 }
 
 
-// Perfect case for a template... (Lgb)
-// or perhaps not...
-#if 1
-template<class T>
-string const write_attribute(string const & name, T const & t)
-{
-       string str = " " + name + "=\"" + tostr(t) + "\"";
-       return str;
-}
-
-template <>
-string const write_attribute(string const & name, bool const & b)
-{
-       return write_attribute(name, int(b));
-}
-
-template <>
-string const write_attribute(string const & name, LyXLength const & value)
-{
-       return write_attribute(name, value.asString());
-}
-#else
-
-string const write_attribute(string const & name, int value)
-{
-       string str = " " + name + "=\"" + tostr(value) + "\"";
-       return str;
-}
-
-
-string const write_attribute(string const & name, string const & value)
-{
-       string str = " " + name + "=\"" + value + "\"";
-       return str;
-}
-
-
-string const write_attribute(string const & name, bool value)
-{
-       string str = " " + name + "=\"" + tostr(static_cast<int>(value)) + "\"";
-       return str;
-}
-
-
-string const write_attribute(string const & name, LyXLength const & value)
-{
-       string str = " " + name + "=\"" + value.asString() + "\"";
-       return str;
-}
-#endif
-
-
-template<>
-inline
-string const tostr(LyXAlignment const & num)
-{
-       switch(num) {
-       case LYX_ALIGN_NONE:
-               return "none";
-       case LYX_ALIGN_BLOCK:
-               return "block";
-       case LYX_ALIGN_LEFT:
-               return "left";
-       case LYX_ALIGN_CENTER:
-               return "center";
-       case LYX_ALIGN_RIGHT:
-               return "right";
-       case LYX_ALIGN_LAYOUT:
-               return "layout";
-       case LYX_ALIGN_SPECIAL:
-               return "special";
-       }
-       return string();
-}
-
-
-template<>
-inline
-string const tostr(LyXTabular::VAlignment const & num)
-{
-       switch(num) {
-       case LyXTabular::LYX_VALIGN_TOP:
-               return "top";
-       case LyXTabular::LYX_VALIGN_CENTER:
-               return "center";
-       case LyXTabular::LYX_VALIGN_BOTTOM:
-               return "bottom";
-       }
-       return string();
-}
-
-
-template<>
-inline
-string const tostr(LyXTabular::BoxType const & num)
-{
-       switch(num) {
-       case LyXTabular::BOX_NONE:
-               return "none";
-       case LyXTabular::BOX_PARBOX:
-               return "parbox";
-       case LyXTabular::BOX_MINIPAGE:
-               return "minipage";
-       }
-       return string();
-}
-
-
 void LyXTabular::Write(Buffer const * buf, ostream & os) const
 {
        // header line
@@ -1117,50 +1036,50 @@ void LyXTabular::Write(Buffer const * buf, ostream & os) const
           << ">\n";
        // global longtable options
        os << "<features"
-          << write_attribute("rotate", tostr(rotate))
-          << write_attribute("islongtable", tostr(is_long_tabular))
-          << write_attribute("firstHeadTopDL", tostr(endfirsthead.topDL))
-          << write_attribute("firstHeadBottomDL", tostr(endfirsthead.bottomDL))
-          << write_attribute("firstHeadEmpty", tostr(endfirsthead.empty))
-          << write_attribute("headTopDL", tostr(endhead.topDL))
-          << write_attribute("headBottomDL", tostr(endhead.bottomDL))
-          << write_attribute("footTopDL", tostr(endfoot.topDL))
-          << write_attribute("footBottomDL", tostr(endfoot.bottomDL))
-          << write_attribute("lastFootTopDL", tostr(endlastfoot.topDL))
-          << write_attribute("lastFootBottomDL", tostr(endlastfoot.bottomDL))
-          << write_attribute("lastFootEmpty", tostr(endlastfoot.empty))
+          << write_attribute("rotate", rotate)
+          << write_attribute("islongtable", is_long_tabular)
+          << write_attribute("firstHeadTopDL", endfirsthead.topDL)
+          << write_attribute("firstHeadBottomDL", endfirsthead.bottomDL)
+          << write_attribute("firstHeadEmpty", endfirsthead.empty)
+          << write_attribute("headTopDL", endhead.topDL)
+          << write_attribute("headBottomDL", endhead.bottomDL)
+          << write_attribute("footTopDL", endfoot.topDL)
+          << write_attribute("footBottomDL", endfoot.bottomDL)
+          << write_attribute("lastFootTopDL", endlastfoot.topDL)
+          << write_attribute("lastFootBottomDL", endlastfoot.bottomDL)
+          << write_attribute("lastFootEmpty", endlastfoot.empty)
           << ">\n";
        for (int j = 0; j < columns_; ++j) {
                os << "<column"
-                  << write_attribute("alignment", tostr(column_info[j].alignment))
-                  << write_attribute("valignment", tostr(column_info[j].valignment))
-                  << write_attribute("leftline", tostr(column_info[j].left_line))
-                  << write_attribute("rightline", tostr(column_info[j].right_line))
+                  << write_attribute("alignment", column_info[j].alignment)
+                  << write_attribute("valignment", column_info[j].valignment)
+                  << write_attribute("leftline", column_info[j].left_line)
+                  << write_attribute("rightline", column_info[j].right_line)
                   << write_attribute("width", column_info[j].p_width.asString())
                   << write_attribute("special", column_info[j].align_special)
                   << ">\n";
        }
        for (int i = 0; i < rows_; ++i) {
                os << "<row"
-                  << write_attribute("topline", tostr(row_info[i].top_line))
-                  << write_attribute("bottomline", tostr(row_info[i].bottom_line))
-                  << write_attribute("endhead", tostr(row_info[i].endhead))
-                  << write_attribute("endfirsthead", tostr(row_info[i].endfirsthead))
-                  << write_attribute("endfoot", tostr(row_info[i].endfoot))
-                  << write_attribute("endlastfoot", tostr(row_info[i].endlastfoot))
-                  << write_attribute("newpage", tostr(row_info[i].newpage))
+                  << write_attribute("topline", row_info[i].top_line)
+                  << write_attribute("bottomline", row_info[i].bottom_line)
+                  << write_attribute("endhead", row_info[i].endhead)
+                  << write_attribute("endfirsthead", row_info[i].endfirsthead)
+                  << write_attribute("endfoot", row_info[i].endfoot)
+                  << write_attribute("endlastfoot", row_info[i].endlastfoot)
+                  << write_attribute("newpage", row_info[i].newpage)
                   << ">\n";
                for (int j = 0; j < columns_; ++j) {
                        os << "<cell"
                           << write_attribute("multicolumn", cell_info[i][j].multicolumn)
-                          << write_attribute("alignment", tostr(cell_info[i][j].alignment))
-                          << write_attribute("valignment", tostr(cell_info[i][j].valignment))
-                          << write_attribute("topline", tostr(cell_info[i][j].top_line))
-                          << write_attribute("bottomline", tostr(cell_info[i][j].bottom_line))
-                          << write_attribute("leftline", tostr(cell_info[i][j].left_line))
-                          << write_attribute("rightline", tostr(cell_info[i][j].right_line))
-                          << write_attribute("rotate", tostr(cell_info[i][j].rotate))
-                          << write_attribute("usebox", tostr(cell_info[i][j].usebox))
+                          << write_attribute("alignment", cell_info[i][j].alignment)
+                          << write_attribute("valignment", cell_info[i][j].valignment)
+                          << write_attribute("topline", cell_info[i][j].top_line)
+                          << write_attribute("bottomline", cell_info[i][j].bottom_line)
+                          << write_attribute("leftline", cell_info[i][j].left_line)
+                          << write_attribute("rightline", cell_info[i][j].right_line)
+                          << write_attribute("rotate", cell_info[i][j].rotate)
+                          << write_attribute("usebox", cell_info[i][j].usebox)
                           << write_attribute("width", cell_info[i][j].p_width)
                           << write_attribute("special", cell_info[i][j].align_special)
                           << ">\n";
@@ -1175,165 +1094,6 @@ void LyXTabular::Write(Buffer const * buf, ostream & os) const
 }
 
 
-namespace {
-
-// I would have liked a fromstr template a lot better. (Lgb)
-
-inline
-bool string2type(string const str, LyXAlignment & num)
-{
-       if (str == "none")
-               num = LYX_ALIGN_NONE;
-       else if (str == "block")
-               num = LYX_ALIGN_BLOCK;
-       else if (str == "left")
-               num = LYX_ALIGN_LEFT;
-       else if (str == "center")
-               num = LYX_ALIGN_CENTER;
-       else if (str == "right")
-               num = LYX_ALIGN_RIGHT;
-       else
-               return false;
-       return true;
-}
-
-
-inline
-bool string2type(string const str, LyXTabular::VAlignment & num)
-{
-       if (str == "top")
-               num = LyXTabular::LYX_VALIGN_TOP;
-       else if (str == "center")
-               num = LyXTabular::LYX_VALIGN_CENTER;
-       else if (str == "bottom")
-               num = LyXTabular::LYX_VALIGN_BOTTOM;
-       else
-               return false;
-       return true;
-}
-
-
-inline
-bool string2type(string const str, LyXTabular::BoxType & num)
-{
-       if (str == "none")
-               num = LyXTabular::BOX_NONE;
-       else if (str == "parbox")
-               num = LyXTabular::BOX_PARBOX;
-       else if (str == "minipage")
-               num = LyXTabular::BOX_MINIPAGE;
-       else
-               return false;
-       return true;
-}
-
-
-inline
-bool string2type(string const str, bool & num)
-{
-       if (str == "true")
-               num = true;
-       else if (str == "false")
-               num = false;
-       else
-               return false;
-       return true;
-}
-
-
-bool getTokenValue(string const & str, const char * token, string & ret)
-{
-       size_t token_length = strlen(token);
-       string::size_type pos = str.find(token);
-
-       if (pos == string::npos || pos + token_length + 1 >= str.length()
-               || str[pos + token_length] != '=')
-               return false;
-       ret.erase();
-       pos += token_length + 1;
-       char ch = str[pos];
-       if ((ch != '"') && (ch != '\'')) { // only read till next space
-               ret += ch;
-               ch = ' ';
-       }
-       while ((pos < str.length() - 1) && (str[++pos] != ch))
-               ret += str[pos];
-
-       return true;
-}
-
-
-bool getTokenValue(string const & str, const char * token, int & num)
-{
-       string tmp;
-       if (!getTokenValue(str, token, tmp))
-               return false;
-       num = strToInt(tmp);
-       return true;
-}
-
-
-bool getTokenValue(string const & str, const char * token, LyXAlignment & num)
-{
-       string tmp;
-       if (!getTokenValue(str, token, tmp))
-               return false;
-       return string2type(tmp, num);
-}
-
-
-bool getTokenValue(string const & str, const char * token,
-                                  LyXTabular::VAlignment & num)
-{
-       string tmp;
-       if (!getTokenValue(str, token, tmp))
-               return false;
-       return string2type(tmp, num);
-}
-
-
-bool getTokenValue(string const & str, const char * token,
-                                  LyXTabular::BoxType & num)
-{
-       string tmp;
-       if (!getTokenValue(str, token, tmp))
-               return false;
-       return string2type(tmp, num);
-}
-
-
-bool getTokenValue(string const & str, const char * token, bool & flag)
-{
-       string tmp;
-       if (!getTokenValue(str, token, tmp))
-               return false;
-       return string2type(tmp, flag);
-}    
-
-
-bool getTokenValue(string const & str, const char * token, LyXLength & len)
-{
-       string tmp;
-       if (!getTokenValue(str, token, tmp))
-               return false;
-       return isValidLength(tmp, &len);
-}    
-
-
-inline
-void l_getline(istream & is, string & str)
-{
-       str.erase();
-       while (str.empty()) {
-               getline(is, str);
-               if (!str.empty() && str[str.length() - 1] == '\r')
-                       str.erase(str.length() - 1);
-       }
-}
-
-} // namespace anon
-
-
 void LyXTabular::Read(Buffer const * buf, LyXLex & lex)
 {
        string line;
@@ -1342,7 +1102,7 @@ void LyXTabular::Read(Buffer const * buf, LyXLex & lex)
        l_getline(is, line);
        if (!prefixIs(line, "<lyxtabular ")
                && !prefixIs(line, "<LyXTabular ")) {
-               OldFormatRead(lex, line);
+               OldFormatRead(buf->params, lex, line);
                return;
        }
 
@@ -1355,9 +1115,85 @@ void LyXTabular::Read(Buffer const * buf, LyXLex & lex)
                ReadNew(buf, is, lex, line, version);
 }
 
+void LyXTabular::setHeaderFooterRows(int hr, int fhr, int fr, int lfr)
+{
+       // set header info
+       while(hr > 0) {
+               row_info[--hr].endhead = true;
+       }
+       // set firstheader info
+       if (fhr && (fhr < rows_)) {
+               if (row_info[fhr].endhead) {
+                       while(fhr > 0) {
+                               row_info[--fhr].endfirsthead = true;
+                               row_info[fhr].endhead = false;
+                       }
+               } else if (row_info[fhr - 1].endhead) {
+                       endfirsthead.empty = true;
+               } else {
+                       while((fhr > 0) && !row_info[--fhr].endhead) {
+                               row_info[fhr].endfirsthead = true;
+                       }
+               }
+       }
+       // set footer info
+       if (fr && (fr < rows_)) {
+               if (row_info[fr].endhead && row_info[fr-1].endhead) {
+                       while((fr > 0) && !row_info[--fr].endhead) {
+                               row_info[fr].endfoot = true;
+                               row_info[fr].endhead = false;
+                       }
+               } else if (row_info[fr].endfirsthead && row_info[fr-1].endfirsthead) {
+                       while((fr > 0) && !row_info[--fr].endfirsthead) {
+                               row_info[fr].endfoot = true;
+                               row_info[fr].endfirsthead = false;
+                       }
+               } else if (!row_info[fr - 1].endhead && !row_info[fr - 1].endfirsthead) {
+                       while((fr > 0) && !row_info[--fr].endhead &&
+                                 !row_info[fr].endfirsthead)
+                       {
+                               row_info[fr].endfoot = true;
+                       }
+               }
+       }
+       // set lastfooter info
+       if (lfr && (lfr < rows_)) {
+               if (row_info[lfr].endhead && row_info[lfr - 1].endhead) {
+                       while((lfr > 0) && !row_info[--lfr].endhead) {
+                               row_info[lfr].endlastfoot = true;
+                               row_info[lfr].endhead = false;
+                       }
+               } else if (row_info[lfr].endfirsthead &&
+                                  row_info[lfr - 1].endfirsthead)
+               {
+                       while((lfr > 0) && !row_info[--lfr].endfirsthead) {
+                               row_info[lfr].endlastfoot = true;
+                               row_info[lfr].endfirsthead = false;
+                       }
+               } else if (row_info[lfr].endfoot
+                          && row_info[lfr - 1].endfoot) {
+                       while((lfr > 0) && !row_info[--lfr].endfoot) {
+                               row_info[lfr].endlastfoot = true;
+                               row_info[lfr].endfoot = false;
+                       }
+               } else if (!row_info[fr - 1].endhead
+                          && !row_info[fr - 1].endfirsthead &&
+                                  !row_info[fr - 1].endfoot)
+               {
+                       while((lfr > 0) &&
+                                 !row_info[--lfr].endhead && !row_info[lfr].endfirsthead &&
+                                 !row_info[lfr].endfoot)
+                       {
+                               row_info[lfr].endlastfoot = true;
+                       }
+               } else if (haveLTFoot()) {
+                       endlastfoot.empty = true;
+               }
+       }
+}
 
 void LyXTabular::ReadNew(Buffer const * buf, istream & is,
-                                                LyXLex & lex, string const & l, int const version)
+                        LyXLex & lex, string const & l, int const version)
 {
        string line(l);
        int rows_arg;
@@ -1366,16 +1202,23 @@ void LyXTabular::ReadNew(Buffer const * buf, istream & is,
        int columns_arg;
        if (!getTokenValue(line, "columns", columns_arg))
                return;
-       Init(rows_arg, columns_arg);
+       Init(buf->params, rows_arg, columns_arg);
        l_getline(is, line);
        if (!prefixIs(line, "<features")) {
-               lyxerr << "Wrong tabular format (expected <features ...> got" <<
-                       line << ")" << endl;
+               lyxerr << "Wrong tabular format (expected <features ...> got"
+                      << line << ')' << endl;
                return;
        }
        getTokenValue(line, "rotate", rotate);
        getTokenValue(line, "islongtable", is_long_tabular);
-       // compatibility read for old longtable options
+       // compatibility read for old longtable options. Now we can make any
+       // row part of the header/footer type we want before it was strict
+       // sequential from the first row down (as LaTeX does it!). So now when
+       // we find a header/footer line we have to go up the rows and set it
+       // on all preceding rows till the first or one with already a h/f option
+       // set. If we find a firstheader on the same line as a header or a
+       // lastfooter on the same line as a footer then this should be set empty.
+       // (Jug 20011220)
        if (version < 3) {
                int hrow;
                int fhrow;
@@ -1386,14 +1229,7 @@ void LyXTabular::ReadNew(Buffer const * buf, istream & is,
                getTokenValue(line, "endfirsthead", fhrow);
                getTokenValue(line, "endfoot", frow);
                getTokenValue(line, "endlastfoot", lfrow);
-               if (abs(hrow) > 0)
-                       row_info[abs(hrow)-1].endhead = true;
-               if (abs(fhrow))
-                       row_info[abs(fhrow)-1].endfirsthead = true;
-               if (abs(frow))
-                       row_info[abs(frow)-1].endfoot = true;
-               if (abs(lfrow))
-                       row_info[abs(lfrow)-1].endlastfoot = true;
+               setHeaderFooterRows(abs(hrow), abs(fhrow), abs(frow), abs(lfrow));
        } else {
           getTokenValue(line, "firstHeadTopDL", endfirsthead.topDL);
           getTokenValue(line, "firstHeadBottomDL", endfirsthead.bottomDL);
@@ -1409,8 +1245,8 @@ void LyXTabular::ReadNew(Buffer const * buf, istream & is,
        for (int j = 0; j < columns_; ++j) {
                l_getline(is,line);
                if (!prefixIs(line,"<column")) {
-                       lyxerr << "Wrong tabular format (expected <column ...> got" <<
-                               line << ")" << endl;
+                       lyxerr << "Wrong tabular format (expected <column ...> got"
+                              << line << ')' << endl;
                        return;
                }
                getTokenValue(line, "alignment", column_info[j].alignment);
@@ -1424,18 +1260,22 @@ void LyXTabular::ReadNew(Buffer const * buf, istream & is,
        for (int i = 0; i < rows_; ++i) {
                l_getline(is, line);
                if (!prefixIs(line, "<row")) {
-                       lyxerr << "Wrong tabular format (expected <row ...> got" <<
-                               line << ")" << endl;
+                       lyxerr << "Wrong tabular format (expected <row ...> got"
+                              << line << ')' << endl;
                        return;
                }
                getTokenValue(line, "topline", row_info[i].top_line);
                getTokenValue(line, "bottomline", row_info[i].bottom_line);
+               getTokenValue(line, "endfirsthead", row_info[i].endfirsthead);
+               getTokenValue(line, "endhead", row_info[i].endhead);
+               getTokenValue(line, "endfoot", row_info[i].endfoot);
+               getTokenValue(line, "endlastfoot", row_info[i].endlastfoot);
                getTokenValue(line, "newpage", row_info[i].newpage);
                for (int j = 0; j < columns_; ++j) {
                        l_getline(is, line);
                        if (!prefixIs(line, "<cell")) {
-                               lyxerr << "Wrong tabular format (expected <cell ...> got" <<
-                                       line << ")" << endl;
+                               lyxerr << "Wrong tabular format (expected <cell ...> got"
+                                      << line << ')' << endl;
                                return;
                        }
                        getTokenValue(line, "multicolumn", cell_info[i][j].multicolumn);
@@ -1455,15 +1295,15 @@ void LyXTabular::ReadNew(Buffer const * buf, istream & is,
                                l_getline(is, line);
                        }
                        if (!prefixIs(line, "</cell>")) {
-                               lyxerr << "Wrong tabular format (expected </cell> got" <<
-                                       line << ")" << endl;
+                               lyxerr << "Wrong tabular format (expected </cell> got"
+                                      << line << ')' << endl;
                                return;
                        }
                }
                l_getline(is, line);
                if (!prefixIs(line, "</row>")) {
-                       lyxerr << "Wrong tabular format (expected </row> got" <<
-                               line << ")" << endl;
+                       lyxerr << "Wrong tabular format (expected </row> got"
+                              << line << ')' << endl;
                        return;
                }
        }
@@ -1474,7 +1314,8 @@ void LyXTabular::ReadNew(Buffer const * buf, istream & is,
 }
 
 
-void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
+void LyXTabular::OldFormatRead(BufferParams const & bp,
+                              LyXLex & lex, string const & fl)
 {
        int version;
        int i;
@@ -1490,7 +1331,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
        int e = 0;
        int f = 0;
        int g = 0;
-       
+
        istream & is = lex.getStream();
        string s(fl);
        if (s.length() > 8)
@@ -1512,7 +1353,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
                           >> rotate_arg >> a >> b >> c >> d;
                } else
                        is >> rows_arg >> columns_arg;
-               Init(rows_arg, columns_arg);
+               Init(bp, rows_arg, columns_arg);
                cont_row_info = vector<int>(rows_arg);
                SetLongTabular(is_long_tabular_arg);
                SetRotateTabular(rotate_arg);
@@ -1532,14 +1373,11 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
        } else {
                is >> rows_arg >> columns_arg >> is_long_tabular_arg
                   >> rotate_arg >> a >> b >> c >> d;
-               Init(rows_arg, columns_arg);
+               Init(bp, rows_arg, columns_arg);
                cont_row_info = vector<int>(rows_arg);
                SetLongTabular(is_long_tabular_arg);
                SetRotateTabular(rotate_arg);
-               row_info[a].endhead = true;
-               row_info[b].endfirsthead = true;
-               row_info[c].endfoot = true;
-               row_info[d].endlastfoot = true;
+               setHeaderFooterRows(a+1, b+1 , c+1, d+1);
                for (i = 0; i < rows_; ++i) {
                        a = b = c = d = e = f = g = 0;
                        is >> a >> b >> c >> d;
@@ -1558,7 +1396,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
 #else
                        // ignore is buggy but we will use it later (Lgb)
                        is.ignore(); // skip '"'
-#endif    
+#endif
                        getline(is, s1, '"');
 #if 1
                        is >> ch; // skip '"'
@@ -1611,6 +1449,8 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
        Paragraph * par = new Paragraph;
        Paragraph * return_par = 0;
 
+       par->layout(bp.getLyXTextClass().defaultLayout());
+
        string tmptok;
        int pos = 0;
        Paragraph::depth_type depth = 0;
@@ -1623,14 +1463,11 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
                if (token.empty())
                        continue;
                if (token == "\\layout"
-                       || token == "\\end_float"
-                       || token == "\\end_deeper") {
+                       || token == "\\end_float" // this should not exist anymore
+                       || token == "\\end_inset" // as it is substituted by this
+                       || token == "\\end_deeper")
+               {
                        lex.pushToken(token);
-#ifndef NO_COMPABILITY
-                       // Here we need to insert the inset_ert_contents into the last
-                       // cell of the tabular.
-                       owner_->bufferOwner()->insertErtContents(par, pos, font);
-#endif
                        break;
                }
                if (owner_->bufferOwner()->parseSingleLyXformat2Token(lex, par,
@@ -1705,15 +1542,25 @@ LyXTabular::cellstruct * LyXTabular::cellinfo_of_cell(int cell) const
 }
 
 
-void LyXTabular::SetMultiColumn(int cell, int number)
+void LyXTabular::SetMultiColumn(Buffer const * buffer, int cell, int number)
 {
        cellinfo_of_cell(cell)->multicolumn = CELL_BEGIN_OF_MULTICOLUMN;
        cellinfo_of_cell(cell)->alignment = column_info[column_of_cell(cell)].alignment;
        cellinfo_of_cell(cell)->top_line = row_info[row_of_cell(cell)].top_line;
        cellinfo_of_cell(cell)->bottom_line = row_info[row_of_cell(cell)].bottom_line;
+       cellinfo_of_cell(cell)->right_line = column_info[column_of_cell(cell+number-1)].right_line;
+#if 1
+       for (int i = 1; i < number; ++i) {
+               cellinfo_of_cell(cell+i)->multicolumn = CELL_PART_OF_MULTICOLUMN;
+               cellinfo_of_cell(cell)->inset.appendParagraphs(buffer->params,
+                       cellinfo_of_cell(cell+i)->inset.paragraph());
+               cellinfo_of_cell(cell+i)->inset.clear(false);
+       }
+#else
        for (number--; number > 0; --number) {
                cellinfo_of_cell(cell+number)->multicolumn = CELL_PART_OF_MULTICOLUMN;
        }
+#endif
        set_row_column_number_info();
 }
 
@@ -1738,9 +1585,9 @@ int LyXTabular::UnsetMultiColumn(int cell)
 {
        int const row = row_of_cell(cell);
        int column = column_of_cell(cell);
-       
+
        int result = 0;
-       
+
        if (cell_info[row][column].multicolumn == CELL_BEGIN_OF_MULTICOLUMN) {
                cell_info[row][column].multicolumn = CELL_NORMAL;
                ++column;
@@ -1886,14 +1733,19 @@ LyXTabular::BoxType LyXTabular::GetUsebox(int cell) const
 }
 
 
+///
+//  This are functions used for the longtable support
+///
 void LyXTabular::SetLTHead(int row, bool flag, ltType const & hd, bool first)
 {
        if (first) {
                endfirsthead = hd;
-               row_info[row].endfirsthead = flag;
+               if (hd.set)
+                       row_info[row].endfirsthead = flag;
        } else {
                endhead = hd;
-               row_info[row].endhead = flag;
+               if (hd.set)
+                       row_info[row].endhead = flag;
        }
 }
 
@@ -1901,6 +1753,7 @@ void LyXTabular::SetLTHead(int row, bool flag, ltType const & hd, bool first)
 bool LyXTabular::GetRowOfLTHead(int row, ltType & hd) const
 {
        hd = endhead;
+       hd.set = haveLTHead();
        return row_info[row].endhead;
 }
 
@@ -1908,6 +1761,7 @@ bool LyXTabular::GetRowOfLTHead(int row, ltType & hd) const
 bool LyXTabular::GetRowOfLTFirstHead(int row, ltType & hd) const
 {
        hd = endfirsthead;
+       hd.set = haveLTFirstHead();
        return row_info[row].endfirsthead;
 }
 
@@ -1916,10 +1770,12 @@ void LyXTabular::SetLTFoot(int row, bool flag, ltType const & fd, bool last)
 {
        if (last) {
                endlastfoot = fd;
-               row_info[row].endlastfoot = flag;
+               if (fd.set)
+                       row_info[row].endlastfoot = flag;
        } else {
                endfoot = fd;
-               row_info[row].endfoot = flag;
+               if (fd.set)
+                       row_info[row].endfoot = flag;
        }
 }
 
@@ -1927,6 +1783,7 @@ void LyXTabular::SetLTFoot(int row, bool flag, ltType const & fd, bool last)
 bool LyXTabular::GetRowOfLTFoot(int row, ltType & fd) const
 {
        fd = endfoot;
+       fd.set = haveLTFoot();
        return row_info[row].endfoot;
 }
 
@@ -1934,6 +1791,7 @@ bool LyXTabular::GetRowOfLTFoot(int row, ltType & fd) const
 bool LyXTabular::GetRowOfLTLastFoot(int row, ltType & fd) const
 {
        fd = endlastfoot;
+       fd.set = haveLTLastFoot();
        return row_info[row].endlastfoot;
 }
 
@@ -1950,6 +1808,52 @@ bool LyXTabular::GetLTNewPage(int row) const
 }
 
 
+bool LyXTabular::haveLTHead() const
+{
+       for(int i=0; i < rows_; ++i) {
+               if (row_info[i].endhead)
+                       return true;
+       }
+       return false;
+}
+
+
+bool LyXTabular::haveLTFirstHead() const
+{
+       if (endfirsthead.empty)
+               return false;
+       for(int i=0; i < rows_; ++i) {
+               if (row_info[i].endfirsthead)
+                       return true;
+       }
+       return false;
+}
+
+
+bool LyXTabular::haveLTFoot() const
+{
+       for(int i=0; i < rows_; ++i) {
+               if (row_info[i].endfoot)
+                       return true;
+       }
+       return false;
+}
+
+
+bool LyXTabular::haveLTLastFoot() const
+{
+       if (endlastfoot.empty)
+               return false;
+       for(int i=0; i < rows_; ++i) {
+               if (row_info[i].endlastfoot)
+                       return true;
+       }
+       return false;
+}
+
+
+// end longtable support functions
+
 bool LyXTabular::SetAscentOfRow(int row, int height)
 {
        if ((row >= rows_) || (row_info[row].ascent_of_row == height))
@@ -2016,7 +1920,7 @@ int LyXTabular::TeXTopHLine(ostream & os, int row) const
                if (TopLine(i))
                        ++tmp;
        }
-       if (tmp == (n - fcell)){
+       if (tmp == (n - fcell)) {
                os << "\\hline ";
        } else if (tmp) {
                for (int i = fcell; i < n; ++i) {
@@ -2049,7 +1953,7 @@ int LyXTabular::TeXBottomHLine(ostream & os, int row) const
                if (BottomLine(i))
                        ++tmp;
        }
-       if (tmp == (n-fcell)){
+       if (tmp == (n - fcell)) {
                os << "\\hline";
        } else if (tmp) {
                for (int i = fcell; i < n; ++i) {
@@ -2083,7 +1987,7 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
                        os << cellinfo_of_cell(cell)->align_special << "}{";
                } else {
                        if (LeftLine(cell) &&
-                               (IsFirstCellInRow(cell) || 
+                               (IsFirstCellInRow(cell) ||
                                 (!IsMultiColumn(cell-1) && !LeftLine(cell, true) &&
                                  !RightLine(cell-1, true))))
                        {
@@ -2092,16 +1996,18 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
                        if (!GetPWidth(cell).zero()) {
                                switch (GetVAlignment(cell)) {
                                case LYX_VALIGN_TOP:
-                                       os << "p";
+                                       os << 'p';
                                        break;
                                case LYX_VALIGN_CENTER:
-                                       os << "m";
+                                       os << 'm';
                                        break;
                                case LYX_VALIGN_BOTTOM:
-                                       os << "b";
+                                       os << 'b';
                                        break;
                                }
-                               os << "{" << GetPWidth(cell).asLatexString() << '}';
+                               os << '{'
+                                  << GetPWidth(cell).asLatexString()
+                                  << '}';
                        } else {
                                switch (GetAlignment(cell)) {
                                case LYX_ALIGN_LEFT:
@@ -2127,13 +2033,13 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
                os << "\\parbox[";
                switch (GetVAlignment(cell)) {
                case LYX_VALIGN_TOP:
-                       os << "t";
+                       os << 't';
                        break;
                case LYX_VALIGN_CENTER:
-                       os << "c";
+                       os << 'c';
                        break;
                case LYX_VALIGN_BOTTOM:
-                       os << "b";
+                       os << 'b';
                        break;
                }
                os << "]{" << GetPWidth(cell).asLatexString() << "}{";
@@ -2141,13 +2047,13 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
                os << "\\begin{minipage}[";
                switch (GetVAlignment(cell)) {
                case LYX_VALIGN_TOP:
-                       os << "t";
+                       os << 't';
                        break;
                case LYX_VALIGN_CENTER:
-                       os << "m";
+                       os << 'm';
                        break;
                case LYX_VALIGN_BOTTOM:
-                       os << "b";
+                       os << 'b';
                        break;
                }
                os << "]{" << GetPWidth(cell).asLatexString() << "}\n";
@@ -2163,12 +2069,12 @@ int LyXTabular::TeXCellPostamble(ostream & os, int cell) const
 
        // usual cells
        if (GetUsebox(cell) == BOX_PARBOX)
-               os << "}";
+               os << '}';
        else if (GetUsebox(cell) == BOX_MINIPAGE) {
                os << "%\n\\end{minipage}";
                ret += 2;
        }
-       if (IsMultiColumn(cell)){
+       if (IsMultiColumn(cell)) {
                os << '}';
        }
        if (GetRotateCell(cell)) {
@@ -2179,11 +2085,146 @@ int LyXTabular::TeXCellPostamble(ostream & os, int cell) const
 }
 
 
-int LyXTabular::Latex(Buffer const * buf,
+int LyXTabular::TeXLongtableHeaderFooter(ostream & os, Buffer const * buf,
+                                        bool fragile, bool fp) const
+{
+       if (!is_long_tabular)
+               return 0;
+
+       int ret = 0;
+       // output header info
+       if (haveLTHead()) {
+               if (endhead.topDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               for (int i = 0; i < rows_; ++i) {
+                       if (row_info[i].endhead) {
+                               ret += TeXRow(os, i, buf, fragile, fp);
+                       }
+               }
+               if (endhead.bottomDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               os << "\\endhead\n";
+               ++ret;
+               if (endfirsthead.empty) {
+                       os << "\\endfirsthead\n";
+                       ++ret;
+               }
+       }
+       // output firstheader info
+       if (haveLTFirstHead()) {
+               if (endfirsthead.topDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               for (int i = 0; i < rows_; ++i) {
+                       if (row_info[i].endfirsthead) {
+                               ret += TeXRow(os, i, buf, fragile, fp);
+                       }
+               }
+               if (endfirsthead.bottomDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               os << "\\endfirsthead\n";
+               ++ret;
+       }
+       // output footer info
+       if (haveLTFoot()) {
+               if (endfoot.topDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               for (int i = 0; i < rows_; ++i) {
+                       if (row_info[i].endfoot) {
+                               ret += TeXRow(os, i, buf, fragile, fp);
+                       }
+               }
+               if (endfoot.bottomDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               os << "\\endfoot\n";
+               ++ret;
+               if (endlastfoot.empty) {
+                       os << "\\endlastfoot\n";
+                       ++ret;
+               }
+       }
+       // output lastfooter info
+       if (haveLTLastFoot()) {
+               if (endlastfoot.topDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               for (int i = 0; i < rows_; ++i) {
+                       if (row_info[i].endlastfoot) {
+                               ret += TeXRow(os, i, buf, fragile, fp);
+                       }
+               }
+               if (endlastfoot.bottomDL) {
+                       os << "\\hline\n";
+                       ++ret;
+               }
+               os << "\\endlastfoot\n";
+               ++ret;
+       }
+       return ret;
+}
+
+
+bool LyXTabular::isValidRow(int const row) const
+{
+       if (!is_long_tabular)
+               return true;
+       return (!row_info[row].endhead && !row_info[row].endfirsthead &&
+                       !row_info[row].endfoot && !row_info[row].endlastfoot);
+}
+
+
+int LyXTabular::TeXRow(ostream & os, int const i, Buffer const * buf,
+                      bool fragile, bool fp) const
+{
+       int ret = 0;
+       int cell = GetCellNumber(i, 0);
+
+       ret += TeXTopHLine(os, i);
+       for (int j = 0; j < columns_; ++j) {
+               if (IsPartOfMultiColumn(i,j))
+                       continue;
+               ret += TeXCellPreamble(os, cell);
+               InsetText * inset = GetCellInset(cell);
+
+               bool rtl = inset->paragraph()->isRightToLeftPar(buf->params) &&
+                       !inset->paragraph()->empty() && GetPWidth(cell).zero();
+
+               if (rtl)
+                       os << "\\R{";
+               ret += inset->latex(buf, os, fragile, fp);
+               if (rtl)
+                       os << '}';
+
+               ret += TeXCellPostamble(os, cell);
+               if (!IsLastCellInRow(cell)) { // not last cell in row
+                       os << "&\n";
+                       ++ret;
+               }
+               ++cell;
+       }
+       os << "\\tabularnewline\n";
+       ++ret;
+       ret += TeXBottomHLine(os, i);
+       return ret;
+}
+
+
+int LyXTabular::latex(Buffer const * buf,
                                          ostream & os, bool fragile, bool fp) const
 {
        int ret = 0;
-       int cell = 0;
 
        //+---------------------------------------------------------------------
        //+                      first the opening preamble                    +
@@ -2200,22 +2241,39 @@ int LyXTabular::Latex(Buffer const * buf,
        for (int i = 0; i < columns_; ++i) {
                if (!column_info[i].align_special.empty()) {
                        os << column_info[i].align_special;
-               } else { 
+               } else {
                        if (column_info[i].left_line)
                                os << '|';
                        if (!column_info[i].p_width.zero()) {
+                               switch (column_info[i].alignment) {
+                               case LYX_ALIGN_LEFT:
+                                       os << ">{\\raggedright}";
+                                       break;
+                               case LYX_ALIGN_RIGHT:
+                                       os << ">{\\raggedleft}";
+                                       break;
+                               case LYX_ALIGN_CENTER:
+                                       os << ">{\\centering}";
+                                       break;
+                               case LYX_ALIGN_NONE:
+                               case LYX_ALIGN_BLOCK:
+                               case LYX_ALIGN_LAYOUT:
+                               case LYX_ALIGN_SPECIAL:
+                                       break;
+                               }
+
                                switch (column_info[i].valignment) {
                                case LYX_VALIGN_TOP:
-                                       os << "p";
+                                       os << 'p';
                                        break;
                                case LYX_VALIGN_CENTER:
-                                       os << "m";
+                                       os << 'm';
                                        break;
                                case LYX_VALIGN_BOTTOM:
-                                       os << "b";
+                                       os << 'b';
                                        break;
                        }
-                               os << "{"
+                               os << '{'
                                   << column_info[i].p_width.asLatexString()
                                   << '}';
                        } else {
@@ -2224,8 +2282,8 @@ int LyXTabular::Latex(Buffer const * buf,
                                        os << 'l';
                                        break;
                                case LYX_ALIGN_RIGHT:
-                               os << 'r';
-                               break;
+                                       os << 'r';
+                                       break;
                                default:
                                        os << 'c';
                                        break;
@@ -2238,70 +2296,16 @@ int LyXTabular::Latex(Buffer const * buf,
        os << "}\n";
        ++ret;
 
+       ret += TeXLongtableHeaderFooter(os, buf, fragile, fp);
+
        //+---------------------------------------------------------------------
        //+                      the single row and columns (cells)            +
        //+---------------------------------------------------------------------
 
        for (int i = 0; i < rows_; ++i) {
-               ret += TeXTopHLine(os, i);
-#warning Implement top double lines for LT Header/Footers
-#if 0
-               if (ret > bret) {
-                       ret += TeXBottomHLine(os, i-1);
-                       ret += TeXTopHLine(os, i);
-               }
-#endif
-               for (int j = 0; j < columns_; ++j) {
-                       if (IsPartOfMultiColumn(i,j))
-                               continue;
-                       ret += TeXCellPreamble(os, cell);
-                       InsetText * inset = GetCellInset(cell);
-
-                       bool rtl = inset->paragraph()->isRightToLeftPar(buf->params) &&
-                                       inset->paragraph()->size() > 0 && GetPWidth(cell).zero();
-
-                       if (rtl)
-                               os << "\\R{";
-                       ret += inset->latex(buf, os, fragile, fp);
-                       if (rtl)
-                               os << "}";
-
-                       ret += TeXCellPostamble(os, cell);
-                       if (!IsLastCellInRow(cell)) { // not last cell in row
-                               os << "&\n";
-                               ++ret;
-                       }
-                       ++cell;
-               }
-               os << "\\\\\n";
-               ++ret;
-               ret += TeXBottomHLine(os, i);
-               if (IsLongTabular()) {
-                       if (row_info[i].endhead) {
-                               if (endhead.bottomDL)
-                                       ret += TeXBottomHLine(os, i);
-                               os << "\\endhead\n";
-                               ++ret;
-                       }
-                       if (row_info[i].endfirsthead) {
-                               if (endfirsthead.bottomDL)
-                                       ret += TeXBottomHLine(os, i);
-                               os << "\\endfirsthead\n";
-                               ++ret;
-                       }
-                       if (row_info[i].endfoot) {
-                               if (endfoot.bottomDL)
-                                       ret += TeXBottomHLine(os, i);
-                               os << "\\endfoot\n";
-                               ++ret;
-                       }
-                       if (row_info[i].endlastfoot) {
-                               if (endlastfoot.bottomDL)
-                                       ret += TeXBottomHLine(os, i);
-                               os << "\\endlastfoot\n";
-                               ++ret;
-                       }
-                       if (row_info[i].newpage) {
+               if (isValidRow(i)) {
+                       ret += TeXRow(os, i, buf, fragile, fp);
+                       if (is_long_tabular && row_info[i].newpage) {
                                os << "\\newpage\n";
                                ++ret;
                        }
@@ -2329,7 +2333,7 @@ int LyXTabular::docbookRow(Buffer const * buf, ostream & os, int row) const
 {
        int ret = 0;
        int cell = GetFirstCellInRow(row);
-       
+
        os << "<row>\n";
        for (int j = 0; j < columns_; ++j) {
                if (IsPartOfMultiColumn(row, j))
@@ -2359,24 +2363,25 @@ int LyXTabular::docbookRow(Buffer const * buf, ostream & os, int row) const
                case LYX_VALIGN_CENTER:
                        os << "middle";
                }
-               os << "\"";
+               os << '"';
 
                if (IsMultiColumn(cell)) {
                        os << " namest=\"col" << j << "\" ";
-                       os << "nameend=\"col" << j + cells_in_multicolumn(cell) - 1<< "\"";
+                       os << "nameend=\"col" << j + cells_in_multicolumn(cell) - 1<< '"';
                }
 
-               os << ">";
-               ret += GetCellInset(cell)->docbook(buf, os);
+               os << '>';
+               ret += GetCellInset(cell)->docbook(buf, os, true);
                os << "</entry>\n";
                ++cell;
        }
        os << "</row>\n";
        return ret;
-}      
+}
 
 
-int LyXTabular::DocBook(Buffer const * buf, ostream & os) const
+int LyXTabular::docbook(Buffer const * buf, ostream & os,
+                       bool /*mixcont*/) const
 {
        int ret = 0;
 
@@ -2386,7 +2391,7 @@ int LyXTabular::DocBook(Buffer const * buf, ostream & os) const
 
        os << "<tgroup cols=\"" << columns_
           << "\" colsep=\"1\" rowsep=\"1\">\n";
-       
+
        for (int i = 0; i < columns_; ++i) {
                os << "<colspec colname=\"col" << i << "\" align=\"";
                switch (column_info[i].alignment) {
@@ -2408,48 +2413,44 @@ int LyXTabular::DocBook(Buffer const * buf, ostream & os) const
        //+                      Long Tabular case                             +
        //+---------------------------------------------------------------------
 
-#warning Jose please have a look here I changed the longtable header/footer
-#warning ---- options so I had to disable the docbook code (Jug 20011219)
-#if 0
-       if ( IsLongTabular() ) {
-               // Header
-               if(endhead.row || endfirsthead.row ) {
-                       os << "<thead>\n";
-                       if( endfirsthead.row ) {
-                               docbookRow( buf, os, endfirsthead.row - 1);
-                       }
-                       if( endhead.row && endhead.row != endfirsthead.row) {
-                               docbookRow(buf, os, endhead.row - 1);
+       // output header info
+       if (haveLTHead() || haveLTFirstHead()) {
+               os << "<thead>\n";
+               ++ret;
+               for (int i = 0; i < rows_; ++i) {
+                       if (row_info[i].endhead || row_info[i].endfirsthead) {
+                               ret += docbookRow(buf, os, i);
                        }
-                       os << "</thead>\n";
                }
-
-               // Footer
-               if( endfoot.row || endlastfoot.row ) {
-                       os << "<tfoot>\n";
-                       if( endfoot.row ) {
-                               docbookRow( buf, os, endfoot.row - 1);
-                       }
-                       if( endlastfoot.row && endlastfoot.row != endfoot.row) {
-                               docbookRow( buf, os, endlastfoot.row - 1);
+               os << "</thead>\n";
+               ++ret;
+       }
+       // output footer info
+       if (haveLTFoot() || haveLTLastFoot()) {
+               os << "<tfoot>\n";
+               ++ret;
+               for (int i = 0; i < rows_; ++i) {
+                       if (row_info[i].endfoot || row_info[i].endlastfoot) {
+                               ret += docbookRow(buf, os, i);
                        }
-                       os << "</tfoot>\n";
                }
+               os << "</tfoot>\n";
+               ++ret;
        }
-#endif
+
        //+---------------------------------------------------------------------
        //+                      the single row and columns (cells)            +
        //+---------------------------------------------------------------------
 
        os << "<tbody>\n";
+       ++ret;
        for (int i = 0; i < rows_; ++i) {
-               if(!IsLongTabular() || (
-                  !row_info[i].endhead && !row_info[i].endfirsthead &&
-                  !row_info[i].endfoot && !row_info[i].endlastfoot)) {
-                       docbookRow( buf, os, i);
+               if (isValidRow(i)) {
+                       ret += docbookRow(buf, os, i);
                }
        }
        os << "</tbody>\n";
+       ++ret;
        //+---------------------------------------------------------------------
        //+                      the closing of the tabular                    +
        //+---------------------------------------------------------------------
@@ -2460,20 +2461,11 @@ int LyXTabular::DocBook(Buffer const * buf, ostream & os) const
        return ret;
 }
 
-
-namespace {
-
-       inline
-       void print_n_chars(ostream & os, unsigned char ch, int n)
-       {
-               os << string(n, ch);
-       }
-
-} // namespace anon
-
-
-int LyXTabular::AsciiTopHLine(ostream & os, int row,
-                                                         vector<unsigned int> const & clen) const
+//--
+// ASCII export function and helpers
+//--
+int LyXTabular::asciiTopHLine(ostream & os, int row,
+                             vector<unsigned int> const & clen) const
 {
        int const fcell = GetFirstCellInRow(row);
        int const n = NumberOfCellsInRow(fcell) + fcell;
@@ -2504,7 +2496,7 @@ int LyXTabular::AsciiTopHLine(ostream & os, int row,
                int len = clen[column];
                while (IsPartOfMultiColumn(row, ++column))
                        len += clen[column] + 4;
-               print_n_chars(os, ch, len);
+               os << string(len, ch);
                if (TopLine(i)) {
                        if (RightLine(i))
                                os << "-+";
@@ -2519,8 +2511,8 @@ int LyXTabular::AsciiTopHLine(ostream & os, int row,
 }
 
 
-int LyXTabular::AsciiBottomHLine(ostream & os, int row,
-                                                                vector<unsigned int> const & clen) const
+int LyXTabular::asciiBottomHLine(ostream & os, int row,
+                                vector<unsigned int> const & clen) const
 {
        int const fcell = GetFirstCellInRow(row);
        int const n = NumberOfCellsInRow(fcell) + fcell;
@@ -2551,7 +2543,7 @@ int LyXTabular::AsciiBottomHLine(ostream & os, int row,
                int len = clen[column];
                while (IsPartOfMultiColumn(row, ++column))
                        len += clen[column] + 4;
-               print_n_chars(os, ch, len);
+               os << string(len, ch);
                if (BottomLine(i)) {
                        if (RightLine(i))
                                os << "-+";
@@ -2566,13 +2558,19 @@ int LyXTabular::AsciiBottomHLine(ostream & os, int row,
 }
 
 
-int LyXTabular::AsciiPrintCell(Buffer const * buf, ostream & os,
+int LyXTabular::asciiPrintCell(Buffer const * buf, ostream & os,
                               int cell, int row, int column,
-                              vector<unsigned int> const & clen) const
+                              vector<unsigned int> const & clen,
+                              bool onlydata) const
 {
        ostringstream sstr;
        int ret = GetCellInset(cell)->ascii(buf, sstr, 0);
 
+       if (onlydata) {
+               os << sstr.str();
+               return ret;
+       }
+
        if (LeftLine(cell))
                os << "| ";
        else
@@ -2599,11 +2597,9 @@ int LyXTabular::AsciiPrintCell(Buffer const * buf, ostream & os,
                break;
        }
 
-       for (unsigned int i = 0; i < len1; ++i)
-               os << " ";
-       os << sstr.str();
-       for (unsigned int i = 0; i < len2; ++i)
-               os << " ";
+       os << string(len1, ' ')
+          << sstr.str()
+          << string(len2, ' ');
        if (RightLine(cell))
                os << " |";
        else
@@ -2613,7 +2609,8 @@ int LyXTabular::AsciiPrintCell(Buffer const * buf, ostream & os,
 }
 
 
-int LyXTabular::Ascii(Buffer const * buf, ostream & os) const
+int LyXTabular::ascii(Buffer const * buf, ostream & os, int const depth,
+                                         bool onlydata, unsigned char delim) const
 {
        int ret = 0;
 
@@ -2622,49 +2619,68 @@ int LyXTabular::Ascii(Buffer const * buf, ostream & os) const
        //+---------------------------------------------------------------------
        vector<unsigned int> clen(columns_);
 
-       // first all non (real) multicolumn cells!
-       for (int j = 0; j < columns_; ++j) {
-               clen[j] = 0;
-               for (int i = 0; i < rows_; ++i) {
-                       int cell = GetCellNumber(i, j);
-                       if (IsMultiColumn(cell, true))
-                               continue;
-                       ostringstream sstr;
-                       GetCellInset(cell)->ascii(buf, sstr, 0);
-                       if (clen[j] < sstr.str().length())
-                               clen[j] = sstr.str().length();
+       if (!onlydata) {
+               // first all non (real) multicolumn cells!
+               for (int j = 0; j < columns_; ++j) {
+                       clen[j] = 0;
+                       for (int i = 0; i < rows_; ++i) {
+                               int cell = GetCellNumber(i, j);
+                               if (IsMultiColumn(cell, true))
+                                       continue;
+                               ostringstream sstr;
+                               GetCellInset(cell)->ascii(buf, sstr, 0);
+                               if (clen[j] < sstr.str().length())
+                                       clen[j] = sstr.str().length();
+                       }
                }
-       }
-       // then all (real) multicolumn cells!
-       for (int j = 0; j < columns_; ++j) {
-               for (int i = 0; i < rows_; ++i) {
-                       int cell = GetCellNumber(i, j);
-                       if (!IsMultiColumn(cell, true) || IsPartOfMultiColumn(i, j))
-                               continue;
-                       ostringstream sstr;
-                       GetCellInset(cell)->ascii(buf, sstr, 0);
-                       int len = int(sstr.str().length());
-                       int const n = cells_in_multicolumn(cell);
-                       for (int k = j; (len > 0) && (k < (j + n - 1)); ++k)
-                               len -= clen[k];
-                       if (len > int(clen[j + n - 1]))
-                               clen[j + n - 1] = len;
+               // then all (real) multicolumn cells!
+               for (int j = 0; j < columns_; ++j) {
+                       for (int i = 0; i < rows_; ++i) {
+                               int cell = GetCellNumber(i, j);
+                               if (!IsMultiColumn(cell, true) || IsPartOfMultiColumn(i, j))
+                                       continue;
+                               ostringstream sstr;
+                               GetCellInset(cell)->ascii(buf, sstr, 0);
+                               int len = int(sstr.str().length());
+                               int const n = cells_in_multicolumn(cell);
+                               for (int k = j; (len > 0) && (k < (j + n - 1)); ++k)
+                                       len -= clen[k];
+                               if (len > int(clen[j + n - 1]))
+                                       clen[j + n - 1] = len;
+                       }
                }
        }
        int cell = 0;
        for (int i = 0; i < rows_; ++i) {
-               AsciiTopHLine(os, i, clen);
+               if (!onlydata) {
+                       if (asciiTopHLine(os, i, clen)) {
+                               for (int j = 0; j < depth; ++j)
+                                       os << "  ";
+                       }
+               }
                for (int j = 0; j < columns_; ++j) {
                        if (IsPartOfMultiColumn(i,j))
                                continue;
-                       ret += AsciiPrintCell(buf, os, cell, i, j, clen);
+                       if (onlydata && j > 0)
+                               os << delim;
+                       ret += asciiPrintCell(buf, os, cell, i, j, clen, onlydata);
                        ++cell;
                }
                os << endl;
-               AsciiBottomHLine(os, i, clen);
+               if (!onlydata) {
+                       for (int j = 0; j < depth; ++j)
+                               os << "  ";
+                       if (asciiBottomHLine(os, i, clen)) {
+                               for (int j = 0; j < depth; ++j)
+                                       os << "  ";
+                       }
+               }
        }
        return ret;
 }
+//--
+// end ascii export
+//--
 
 
 InsetText * LyXTabular::GetCellInset(int cell) const
@@ -2681,26 +2697,69 @@ InsetText * LyXTabular::GetCellInset(int row, int column) const
 }
 
 
+int LyXTabular::GetCellFromInset(Inset const * inset, int maybe_cell) const
+{
+       // is this inset part of the tabular?
+       if (!inset || inset->owner() != owner_) {
+               lyxerr[Debug::INSETTEXT]
+                       << "this is not a cell of the tabular!" << endl;
+               return -1;
+       }
+
+       const int save_cur_cell = cur_cell;
+       int cell = cur_cell;
+       if (GetCellInset(cell) != inset) {
+               cell = maybe_cell;
+               if (cell == -1 || GetCellInset(cell) != inset) {
+                       cell = -1;
+               }
+       }
+
+       if (cell == -1) {
+               for (cell = GetNumberOfCells(); cell >= 0; --cell) {
+                       if (GetCellInset(cell) == inset)
+                               break;
+               }
+               lyxerr[Debug::INSETTEXT]
+                        << "LyXTabular::GetCellFromInset: "
+                                   << "cell=" << cell
+                                   << ", cur_cell=" << save_cur_cell
+                                   << ", maybe_cell=" << maybe_cell
+                                   << endl;
+               // We should have found a cell at this point
+               if (cell == -1) {
+                       lyxerr << "LyXTabular::GetCellFromInset: "
+                              << "Cell not found!" << endl;
+               }
+       }
+
+       return cell;
+}
+
+
 void LyXTabular::Validate(LaTeXFeatures & features) const
 {
+       features.require("NeedTabularnewline");
        if (IsLongTabular())
                features.require("longtable");
        if (NeedRotating())
                features.require("rotating");
-       for (int cell = 0; !features.isRequired("array") && (cell < numberofcells); ++cell) {
-               if (GetVAlignment(cell) != LYX_VALIGN_TOP)
+       for (int cell = 0; cell < numberofcells; ++cell) {
+               if ( (GetVAlignment(cell) != LYX_VALIGN_TOP) ||
+                    ( !(GetPWidth(cell).zero())&&!(IsMultiColumn(cell)) )
+                  )
                        features.require("array");
                GetCellInset(cell)->validate(features);
        }
 }
 
 
-std::vector<string> const LyXTabular::getLabelList() const
+vector<string> const LyXTabular::getLabelList() const
 {
-       std::vector<string> label_list;
+       vector<string> label_list;
        for (int i = 0; i < rows_; ++i)
                for (int j = 0; j < columns_; ++j) {
-                       std::vector<string> const l =
+                       vector<string> const l =
                                GetCellInset(i, j)->getLabelList();
                        label_list.insert(label_list.end(),
                                          l.begin(), l.end());
@@ -2708,7 +2767,7 @@ std::vector<string> const LyXTabular::getLabelList() const
        return label_list;
 }
 
-                       
+
 LyXTabular::BoxType LyXTabular::UseParbox(int cell) const
 {
        Paragraph * par = GetCellInset(cell)->paragraph();