]> git.lyx.org Git - lyx.git/blobdiff - src/tabular.C
More fixes to insettabular/text (and some missing features added).
[lyx.git] / src / tabular.C
index 215729b7a7cc60eec47a123b7ae9b3b7738b59bd..707c9c353f39c211cde04a86522145f351294d97 100644 (file)
@@ -42,8 +42,6 @@ using std::vector;
 
 static int const WIDTH_OF_LINE = 5;
 
-extern BufferView * current_view;
-
 /// Define a few methods for the inner structs
 
 LyXTabular::cellstruct::cellstruct() 
@@ -94,7 +92,9 @@ LyXTabular::LyXTabular(InsetTabular * inset, LyXTabular const & lt)
 {
     owner_ = inset;
     Init(lt.rows_, lt.columns_);
+#ifdef WITH_WARNINGS
 #warning Jürgen, can you make it the other way round. So that copy assignment depends on the copy constructor and not the other way. (Lgb)
+#endif
     operator=(lt);
 }
 
@@ -106,13 +106,6 @@ LyXTabular::LyXTabular(Buffer const * buf, InsetTabular * inset, LyXLex & lex)
 }
 
 
-LyXTabular::~LyXTabular()
-{
-    delete[] rowofcell;
-    delete[] columnofcell;
-}
-
-
 LyXTabular & LyXTabular::operator=(LyXTabular const & lt)
 {
     // If this and lt is not of the same size we have a serious bug
@@ -144,8 +137,8 @@ LyXTabular * LyXTabular::Clone(InsetTabular * inset)
     LyXTabular * result = new LyXTabular(inset, *this);
     // don't know if this is good but I need to Clone also
     // the text-insets here, this is for the Undo-facility!
-    for(int i = 0; i < rows_; ++i) {
-       for(int j = 0; j < columns_; ++j) {
+    for (int i = 0; i < rows_; ++i) {
+       for (int j = 0; j < columns_; ++j) {
            result->cell_info[i][j].inset = cell_info[i][j].inset;
            result->cell_info[i][j].inset.setOwner(inset);
        }
@@ -157,14 +150,12 @@ LyXTabular * LyXTabular::Clone(InsetTabular * inset)
 /* activates all lines and sets all widths to 0 */ 
 void LyXTabular::Init(int rows_arg, int columns_arg)
 {
-
     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()));
 
-    // Jürgen, use iterators.
     int cellno = 0;
     for (int i = 0; i < rows_; ++i) {
         for (int j = 0; j < columns_; ++j) {
@@ -172,23 +163,20 @@ void LyXTabular::Init(int rows_arg, int columns_arg)
            cell_info[i][j].inset.SetDrawFrame(0, InsetText::LOCKED);
             cell_info[i][j].cellno = cellno++;
         }
-       cell_info[i][columns_-1].right_line = true;
+       cell_info[i].back().right_line = true;
     }
-    //row_info[i - 1].bottom_line = true;
-    //row_info[0].bottom_line = true;
     row_info.back().bottom_line = true;
     row_info.front().bottom_line = true;
 
     for (int i = 0; i < columns_; ++i) {
         calculate_width_of_column(i);
     }
-    //column_info[columns_ - 1].right_line = true;
     column_info.back().right_line = true;
    
     calculate_width_of_tabular();
 
-    rowofcell = 0;
-    columnofcell = 0;
+    rowofcell = vector<int>();
+    columnofcell = vector<int>();
     set_row_column_number_info();
     is_long_tabular = false;
     rotate = false;
@@ -215,13 +203,13 @@ void LyXTabular::AppendRow(int cell)
     cell_vvector c_info = cell_vvector(rows_, cell_vector(columns_,
                                                          cellstruct()));
 
-    for(int i = 0; i <= row; ++i) {
-       for(int j = 0; j < columns_; ++j) {
+    for (int i = 0; i <= row; ++i) {
+       for (int j = 0; j < columns_; ++j) {
            c_info[i][j] = cell_info[i][j];
        }
     }
-    for(int i = row + 1; i < rows_; ++i) {
-       for(int j = 0; j < columns_; ++j) {
+    for (int i = row + 1; i < rows_; ++i) {
+       for (int j = 0; j < columns_; ++j) {
            c_info[i][j] = cell_info[i-1][j];
        }
     }
@@ -255,7 +243,7 @@ void LyXTabular::AppendColumn(int cell)
    
     cell_vvector c_info = cell_vvector(rows_, cell_vector(columns_,
                                                          cellstruct()));
-    int column = column_of_cell(cell);
+    int const column = column_of_cell(cell);
     column_vector::iterator cit = column_info.begin() + column + 1;
     column_info.insert(cit, columnstruct());
 
@@ -276,9 +264,10 @@ void LyXTabular::AppendColumn(int cell)
         }
     }
     cell_info = c_info;
-    ++column;
+    //++column;
     for (int i = 0; i < rows_; ++i) {
-       cell_info[i][column].inset.clear();
+       //cell_info[i][column].inset.clear();
+       cell_info[i][column + 1].inset.clear();
     }
     Reinit();
 }
@@ -302,7 +291,6 @@ void LyXTabular::DeleteColumn(int column)
 
 void LyXTabular::Reinit()
 {   
-    // Jürgen, use iterators.
     for (int i = 0; i < rows_; ++i) {
        for (int j = 0; j < columns_; ++j) {
            cell_info[i][j].width_of_cell = 0;
@@ -321,12 +309,9 @@ void LyXTabular::Reinit()
 
 void LyXTabular::set_row_column_number_info(bool oldformat)
 {
-    int c = 0;
-    int column = 0;
     numberofcells = -1;
-    int row = 0;
-    for (; row < rows_; ++row) {
-       for (column = 0; column<columns_; ++column) {
+    for (int row = 0; row < rows_; ++row) {
+       for (int column = 0; column<columns_; ++column) {
            if (cell_info[row][column].multicolumn
                != LyXTabular::CELL_PART_OF_MULTICOLUMN)
                ++numberofcells;
@@ -334,15 +319,12 @@ void LyXTabular::set_row_column_number_info(bool oldformat)
        }
     }
     ++numberofcells; // because this is one more than as we start from 0
-    row = 0;
-    column = 0;
 
-    delete [] rowofcell;
-    rowofcell = new int[numberofcells];
-    delete [] columnofcell;
-    columnofcell = new int[numberofcells];
-  
-    while (c < numberofcells && row < rows_ && column < columns_) {
+    rowofcell.resize(numberofcells);
+    columnofcell.resize(numberofcells);
+
+    for (int row = 0, column = 0, c = 0;
+        c < numberofcells && row < rows_ && column < columns_;) {
        rowofcell[c] = row;
        columnofcell[c] = column;
        ++c;
@@ -356,8 +338,9 @@ void LyXTabular::set_row_column_number_info(bool oldformat)
            ++row;
        }
     }
-    for (row = 0; row < rows_; ++row) {
-       for (column = 0; column<columns_; ++column) {
+
+    for (int row = 0; row < rows_; ++row) {
+       for (int column = 0; column < columns_; ++column) {
            if (IsPartOfMultiColumn(row,column))
                continue;
            // now set the right line of multicolumns right for oldformat read
@@ -409,7 +392,7 @@ bool LyXTabular::BottomLine(int cell, bool onlycolumn) const
     // no bottom line underneath non-existent cells if you please
     // Isn't that a programming error? Is so this should
     // be an Assert instead. (Lgb)
-    if(cell >= numberofcells)
+    if (cell >= numberofcells)
        return false;
 
     if (!onlycolumn && IsMultiColumn(cell))
@@ -528,8 +511,7 @@ int LyXTabular::GetWidthOfColumn(int cell) const
     int const column1 = column_of_cell(cell);
     int const column2 = right_column_of_cell(cell);
     int result = 0;
-    int i = column1;
-    for (; i <= column2; ++i) {
+    for (int i = column1; i <= column2; ++i) {
        result += column_info[i].width_of_column;
     }
     return result;
@@ -553,15 +535,14 @@ bool LyXTabular::SetWidthOfMulticolCell(int cell, int new_width)
     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) {
+    for (int i = column1; i <= column2; ++i) {
         cell_info[row][i].width_of_cell = 0;
     }
     // set the width to MAX_WIDTH until width > 0
     int width = (new_width + 2 * WIDTH_OF_LINE);
-    for (i = column1;
-        i < column2 && width > column_info[i].width_of_column;
-        ++i) {
+
+    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;
         width -= column_info[i].width_of_column;
     }
@@ -582,7 +563,7 @@ void LyXTabular::recalculateMulticolCells(int cell, int new_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)
+    for (i = cell + 1; (i < numberofcells) && (!IsMultiColumn(i)); ++i)
         ;
     if (i < numberofcells)
         recalculateMulticolCells(i, GetWidthOfCell(i) - (2 * WIDTH_OF_LINE));
@@ -612,13 +593,13 @@ bool LyXTabular::SetWidthOfCell(int cell, int new_width)
     }
     if (tmp) {
         int i = 0;
-        for(; i<columns_; ++i)
+        for (; i<columns_; ++i)
             calculate_width_of_column_NMC(i);
-        for(i = 0; (i < numberofcells) && !IsMultiColumn(i); ++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 (i = 0; i < columns_; ++i)
             calculate_width_of_column(i);
         calculate_width_of_tabular();
         return true;
@@ -655,7 +636,7 @@ bool LyXTabular::SetColumnPWidth(int cell, string const & width)
     column_info[j].p_width = width;
     if (flag) // do this only if there is a width
        SetAlignment(cell, LYX_ALIGN_LEFT);
-    for(int i = 0; i < rows_; ++i) {
+    for (int i = 0; i < rows_; ++i) {
        int c = GetCellNumber(i, j);
        flag = !GetPWidth(c).empty(); // because of multicolumns!
        GetCellInset(c)->SetAutoBreakRows(flag);
@@ -749,7 +730,8 @@ LyXAlignment LyXTabular::GetAlignment(int cell, bool onlycolumn) const
 }
 
 
-LyXTabular::VAlignment LyXTabular::GetVAlignment(int cell, bool onlycolumn) const
+LyXTabular::VAlignment
+LyXTabular::GetVAlignment(int cell, bool onlycolumn) const
 {
     if (!onlycolumn && IsMultiColumn(cell))
        return cellinfo_of_cell(cell)->valignment;
@@ -894,7 +876,7 @@ void LyXTabular::calculate_width_of_tabular()
 int LyXTabular::row_of_cell(int cell) const
 {
     if (cell >= numberofcells)
-        return rows_-1;
+        return rows_ - 1;
     else if (cell < 0)
         return 0;
     return rowofcell[cell];
@@ -904,7 +886,7 @@ int LyXTabular::row_of_cell(int cell) const
 int LyXTabular::column_of_cell(int cell) const
 {
     if (cell >= numberofcells)
-        return columns_-1;
+        return columns_ - 1;
     else if (cell < 0)
         return 0;
     return columnofcell[cell];
@@ -916,82 +898,102 @@ int LyXTabular::right_column_of_cell(int cell) const
     int const row = row_of_cell(cell);
     int column = column_of_cell(cell);
     while (column < (columns_ - 1) &&
-          cell_info[row][column+1].multicolumn == LyXTabular::CELL_PART_OF_MULTICOLUMN)
+          cell_info[row][column + 1].multicolumn == LyXTabular::CELL_PART_OF_MULTICOLUMN)
        ++column;
     return column;
 }
 
 
-const string write_attribute(const string name, const int value)
+// Perfect case for a template... (Lgb)
+#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));
+}
+
+#else
+
+string const write_attribute(string const & name, int value)
 {
     string str = " " + name + "=\"" + tostr(value) + "\"";
     return str;
 }
 
 
-const string write_attribute(string name, const string & value)
+string const write_attribute(string const & name, string const & value)
 {
     string str = " " + name + "=\"" + value + "\"";
     return str;
 }
 
 
-const string write_attribute(string name, const bool value)
+string const write_attribute(string const & name, bool value)
 {
-    string str = " " + name + "=\"" + tostr((int)value) + "\"";
+    string str = " " + name + "=\"" + tostr(static_cast<int>(value)) + "\"";
     return str;
 }
+#endif
 
 
 void LyXTabular::Write(Buffer const * buf, ostream & os) const
 {
     // header line
-    os << "<LyXTabular" <<
-       write_attribute("version", 1) <<
-       write_attribute("rows", rows_) <<
-       write_attribute("columns", columns_) <<
-       ">\n";
+    os << "<LyXTabular"
+       << write_attribute("version", 1)
+       << write_attribute("rows", rows_)
+       << write_attribute("columns", columns_)
+       << ">\n";
     // global longtable options
-    os << "<Features" <<
-       write_attribute("rotate", rotate) <<
-       write_attribute("islongtable", is_long_tabular) <<
-       write_attribute("endhead", endhead) <<
-       write_attribute("endfirsthead", endfirsthead) <<
-       write_attribute("endfoot", endfoot) <<
-       write_attribute("endlastfoot", endlastfoot) <<
-       ">\n\n";
+    os << "<Features"
+       << write_attribute("rotate", rotate)
+       << write_attribute("islongtable", is_long_tabular)
+       << write_attribute("endhead", endhead)
+       << write_attribute("endfirsthead", endfirsthead)
+       << write_attribute("endfoot", endfoot)
+       << write_attribute("endlastfoot", endlastfoot)
+       << ">\n\n";
     for (int i = 0; i < rows_; ++i) {
-       os << "<Row" <<
-           write_attribute("topline", row_info[i].top_line) <<
-           write_attribute("bottomline", row_info[i].bottom_line) <<
-           write_attribute("newpage", row_info[i].newpage) <<
-           ">\n";
+       os << "<Row"
+          << write_attribute("topline", row_info[i].top_line)
+          << write_attribute("bottomline", row_info[i].bottom_line)
+          << write_attribute("newpage", row_info[i].newpage)
+          << ">\n";
        for (int j = 0; j < columns_; ++j) {
            if (!i) {
-               os << "<Column" <<
-                   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", VSpace(column_info[j].p_width).asLyXCommand()) <<
-                   write_attribute("special", column_info[j].align_special) <<
-                   ">\n";
+               os << "<Column"
+                  << 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",
+                                     VSpace(column_info[j].p_width)
+                                     .asLyXCommand())
+                  << write_attribute("special", column_info[j].align_special)
+                  << ">\n";
            } else {
                os << "<Column>\n";
            }
-           os << "<Cell" <<
-               write_attribute("multicolumn", cell_info[i][j].multicolumn) <<
-               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", (int)cell_info[i][j].usebox) <<
-               write_attribute("width", cell_info[i][j].p_width) <<
-               write_attribute("special", cell_info[i][j].align_special) <<
-               ">\n";
+           os << "<Cell"
+              << write_attribute("multicolumn", cell_info[i][j].multicolumn)
+              << 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";
            os << "\\begin_inset ";
            cell_info[i][j].inset.Write(buf, os);
            os << "\n\\end_inset \n"
@@ -1005,21 +1007,21 @@ void LyXTabular::Write(Buffer const * buf, ostream & os) const
 
 
 static
-bool getTokenValue(string const str, const char * token, string & ret)
+bool getTokenValue(string const str, const char * token, string & ret)
 {
-    int pos = str.find(token);
-    char ch = str[pos+strlen(token)];
+    string::size_type pos = str.find(token);
+    char ch = str[pos + strlen(token)];
 
-    if ((pos < 0) || (ch != '='))
+    if ((pos == string::npos) || (ch != '='))
        return false;
     ret.erase();
-    pos += strlen(token)+1;
+    pos += strlen(token) + 1;
     ch = str[pos];
     if ((ch != '"') && (ch != '\'')) { // only read till next space
        ret += ch;
        ch = ' ';
     }
-    while((pos < int(str.length()-1)) && (str[++pos] != ch))
+    while((pos < str.length() - 1) && (str[++pos] != ch))
        ret += str[pos];
 
     return true;
@@ -1027,16 +1029,15 @@ bool getTokenValue(string const str, const char * token, string & ret)
 
 
 static
-bool getTokenValue(string const str, const char * token, int & num)
+bool getTokenValue(string const str, const char * token, int & num)
 {
-    int pos = str.find(token);
-    char ch = str[pos+strlen(token)];
+    string::size_type pos = str.find(token);
+    char ch = str[pos + strlen(token)];
 
-    if ((pos < 0) || (ch != '='))
+    if ((pos == string::npos) || (ch != '='))
        return false;
     string ret;
-    //ret.erase(); // why? (Lgb)
-    pos += strlen(token)+1;
+    pos += strlen(token) + 1;
     ch = str[pos];
     if ((ch != '"') && (ch != '\'')) { // only read till next space
        if (!isdigit(ch))
@@ -1044,7 +1045,7 @@ bool getTokenValue(string const str, const char * token, int & num)
        ret += ch;
     }
     ++pos;
-    while((pos < int(str.length()-1)) && isdigit(str[pos]))
+    while((pos < str.length() - 1) && isdigit(str[pos]))
        ret += str[pos++];
 
     num = strToInt(ret);
@@ -1053,28 +1054,28 @@ bool getTokenValue(string const str, const char * token, int & num)
 
 
 static
-bool getTokenValue(string const str, const char * token, LyXAlignment & num)
+bool getTokenValue(string const str, const char * token, LyXAlignment & num)
 {
     int tmp;
-    bool ret = getTokenValue(str, token, tmp);
+    bool const ret = getTokenValue(str, token, tmp);
     num = static_cast<LyXAlignment>(tmp);
     return ret;
 }
 
 
 static
-bool getTokenValue(string const str, const char * token,
+bool getTokenValue(string const str, const char * token,
                   LyXTabular::VAlignment & num)
 {
     int tmp;
-    bool ret = getTokenValue(str, token, tmp);
+    bool const ret = getTokenValue(str, token, tmp);
     num = static_cast<LyXTabular::VAlignment>(tmp);
     return ret;
 }
 
 
 static
-bool getTokenValue(string const str, const char * token,
+bool getTokenValue(string const str, const char * token,
                   LyXTabular::BoxType & num)
 {
     int tmp;
@@ -1085,16 +1086,15 @@ bool getTokenValue(string const str, const char * token,
 
 
 static
-bool getTokenValue(string const str, const char * token, bool & flag)
+bool getTokenValue(string const str, const char * token, bool & flag)
 {
-    int pos = str.find(token);
-    char ch = str[pos+strlen(token)];
+    string::size_type pos = str.find(token);
+    char ch = str[pos + strlen(token)];
 
-    if ((pos < 0) || (ch != '='))
+    if ((pos == string::npos) || (ch != '='))
        return false;
     string ret;
-    //ret.erase(); // Why? (Lgb)
-    pos += strlen(token)+1;
+    pos += strlen(token) + 1;
     ch = str[pos];
     if ((ch != '"') && (ch != '\'')) { // only read till next space
        if (!isdigit(ch))
@@ -1102,7 +1102,7 @@ bool getTokenValue(string const str, const char * token, bool & flag)
        ret += ch;
     }
     ++pos;
-    while((pos < int(str.length()-1)) && isdigit(str[pos]))
+    while((pos < str.length() - 1) && isdigit(str[pos]))
        ret += str[pos++];
 
     flag = strToInt(ret);
@@ -1151,8 +1151,8 @@ void LyXTabular::Read(Buffer const * buf, LyXLex & lex)
     getTokenValue(line, "endfirsthead", endfirsthead);
     getTokenValue(line, "endfoot", endfoot);
     getTokenValue(line, "endlastfoot", endlastfoot);
-    int i, j;
-    for(i = 0; i < rows_; ++i) {
+
+    for (int i = 0; i < rows_; ++i) {
        l_getline(is, line);
        if (!prefixIs(line, "<Row ")) {
            lyxerr << "Wrong tabular format (expected <Row ...> got" <<
@@ -1162,7 +1162,7 @@ void LyXTabular::Read(Buffer const * buf, LyXLex & lex)
        getTokenValue(line, "topline", row_info[i].top_line);
        getTokenValue(line, "bottomline", row_info[i].bottom_line);
        getTokenValue(line, "newpage", row_info[i].newpage);
-       for (j = 0; j < columns_; ++j) {
+       for (int j = 0; j < columns_; ++j) {
            l_getline(is,line);
            if (!prefixIs(line,"<Column")) {
                lyxerr << "Wrong tabular format (expected <Column ...> got" <<
@@ -1240,7 +1240,6 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
     int e = 0;
     int f = 0;
     int g = 0;
-    int h = 0;
        
     istream & is = lex.getStream();
     string s(fl);
@@ -1292,7 +1291,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
        endfoot = c;
        endlastfoot = d;
        for (i = 0; i < rows_; ++i) {
-           a = b = c = d = e = f = g = h = 0;
+           a = b = c = d = e = f = g = 0;
            is >> a >> b >> c >> d;
            row_info[i].top_line = a;
            row_info[i].bottom_line = b;
@@ -1303,10 +1302,20 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
            string s1;
            string s2;
            is >> a >> b >> c;
+#if 1
            char ch; // skip '"'
            is >> ch;
+#else
+           // ignore is buggy but we will use it later (Lgb)
+           is.ignore(); // skip '"'
+#endif    
            getline(is, s1, '"');
+#if 1
            is >> ch; // skip '"'
+#else
+           // ignore is buggy but we will use it later (Lgb)
+           is.ignore(); // skip '"'
+#endif
            getline(is, s2, '"');
            column_info[i].alignment = static_cast<LyXAlignment>(a);
            column_info[i].left_line = b;
@@ -1319,10 +1328,20 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
                string s1;
                string s2;
                is >> a >> b >> c >> d >> e >> f >> g;
+#if 1
                char ch;
                is >> ch; // skip '"'
+#else
+               // ignore is buggy but we will use it later (Lgb)
+               is.ignore(); // skip '"'
+#endif
                getline(is, s1, '"');
+#if 1
                is >> ch; // skip '"'
+#else
+               // ignore is buggy but we will use it later (Lgb)
+               is.ignore(); // skip '"'
+#endif
                getline(is, s2, '"');
                cell_info[i][j].multicolumn = static_cast<char>(a);
                cell_info[i][j].alignment = static_cast<LyXAlignment>(b);
@@ -1349,6 +1368,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
     int pos = 0;
     char depth = 0;
     LyXFont font(LyXFont::ALL_SANE);
+    font.setLanguage(owner_->BufferOwner()->GetLanguage());
 
     while (lex.IsOK()) {
         lex.nextToken();
@@ -1389,7 +1409,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
     InsetText * inset = GetCellInset(cell);
     int row;
 
-    for(int i = 0; i < par->Last(); ++i) {
+    for (int i = 0; i < par->Last(); ++i) {
        if (par->IsNewline(i)) {
            ++cell;
            if (cell > GetNumberOfCells()) {
@@ -1417,7 +1437,7 @@ void LyXTabular::OldFormatRead(LyXLex & lex, string const & fl)
                par->InsertChar(i, ' ');
            }
        }
-       par->CopyIntoMinibuffer(owner_->BufferOwner()->params, i);
+       par->CopyIntoMinibuffer(*owner_->BufferOwner(), i);
        inset->par->InsertFromMinibuffer(inset->par->Last());
     }
     delete par;
@@ -1903,7 +1923,7 @@ int LyXTabular::GetHeightOfTabular() const
 {
     int height = 0;
 
-    for(int row = 0; row < rows_; ++row)
+    for (int row = 0; row < rows_; ++row)
         height += GetAscentOfRow(row) + GetDescentOfRow(row) +
            GetAdditionalHeight(GetCellNumber(row, 0));
     return height;
@@ -1914,7 +1934,7 @@ bool LyXTabular::IsPartOfMultiColumn(int row, int column) const
 {
     if ((row >= rows_) || (column >= columns_))
         return false;
-    return (cell_info[row][column].multicolumn==CELL_PART_OF_MULTICOLUMN);
+    return (cell_info[row][column].multicolumn == CELL_PART_OF_MULTICOLUMN);
 }
 
 
@@ -1943,7 +1963,7 @@ int LyXTabular::TeXTopHLine(ostream & os, int row) const
     } else {
        return 0;
     }
-    os << endl;
+    os << "\n";
     return 1;
 }
 
@@ -1973,7 +1993,7 @@ int LyXTabular::TeXBottomHLine(ostream & os, int row) const
     } else {
        return 0;
     }
-    os << endl;
+    os << "\n";
     return 1;
 }
 
@@ -1983,18 +2003,18 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
     int ret = 0;
 
     if (GetRotateCell(cell)) {
-       os << "\\begin{sideways}" << endl;
+       os << "\\begin{sideways}\n";
        ++ret;
     }
     if (IsMultiColumn(cell)) {
        os << "\\multicolumn{" << cells_in_multicolumn(cell) << "}{";
-       if (!cellinfo_of_cell(cell+1)->align_special.empty()) {
-           os << cellinfo_of_cell(cell+1)->align_special << "}{";
+       if (!cellinfo_of_cell(cell)->align_special.empty()) {
+           os << cellinfo_of_cell(cell)->align_special << "}{";
        } else {
            if (LeftLine(cell))
                os << '|';
            if (!GetPWidth(cell).empty()) {
-               switch(GetVAlignment(cell)) {
+               switch (GetVAlignment(cell)) {
                case LYX_VALIGN_TOP:
                    os << "p";
                    break;
@@ -2029,7 +2049,7 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
     }
     if (GetUsebox(cell) == BOX_PARBOX) {
        os << "\\parbox[";
-       switch(GetVAlignment(cell)) {
+       switch (GetVAlignment(cell)) {
        case LYX_VALIGN_TOP:
            os << "t";
            break;
@@ -2043,7 +2063,7 @@ int LyXTabular::TeXCellPreamble(ostream & os, int cell) const
        os << "]{" << GetPWidth(cell) << "}{";
     } else if (GetUsebox(cell) == BOX_MINIPAGE) {
        os << "\\begin{minipage}[";
-       switch(GetVAlignment(cell)) {
+       switch (GetVAlignment(cell)) {
        case LYX_VALIGN_TOP:
            os << "t";
            break;
@@ -2094,7 +2114,7 @@ int LyXTabular::Latex(Buffer const * buf,
     //+---------------------------------------------------------------------
 
     if (rotate) {
-       os << "\\begin{sideways}" << endl;
+       os << "\\begin{sideways}\n";
        ++ret;
     }
     if (is_long_tabular)
@@ -2107,7 +2127,7 @@ int LyXTabular::Latex(Buffer const * buf,
        if (!column_info[i].align_special.empty()) {
            os << column_info[i].align_special;
        } else if (!column_info[i].p_width.empty()) {
-           switch(column_info[i].valignment) {
+           switch (column_info[i].valignment) {
            case LYX_VALIGN_TOP:
                os << "p";
                break;
@@ -2137,17 +2157,17 @@ int LyXTabular::Latex(Buffer const * buf,
        if (column_info[i].right_line)
            os << '|';
     }
-    os << "}" << endl;
+    os << "}\n";
     ++ret;
 
     //+---------------------------------------------------------------------
     //+                      the single row and columns (cells)            +
     //+---------------------------------------------------------------------
 
-    int bret;
-    for(int i = 0; i < rows_; ++i) {
+    //int bret;
+    for (int i = 0; i < rows_; ++i) {
        ret += TeXTopHLine(os, i);
-       bret = ret;
+       int bret = ret;
        if (IsLongTabular()) {
            if ((endhead < 0) && (i == (abs(endhead)-1))) {
                os << "\\endhead\n";
@@ -2171,19 +2191,19 @@ int LyXTabular::Latex(Buffer const * buf,
                ret += TeXBottomHLine(os, i-1);
            ret += TeXTopHLine(os, i);
        }
-       for(int j = 0; j < columns_; ++j) {
+       for (int j = 0; j < columns_; ++j) {
            if (IsPartOfMultiColumn(i,j))
                continue;
            ret += TeXCellPreamble(os, cell);
            ret += GetCellInset(cell)->Latex(buf, os, fragile, fp);
            ret += TeXCellPostamble(os, cell);
            if (!IsLastCellInRow(cell)) { // not last cell in row
-               os << "&" << endl;
+               os << "&\n";
                ++ret;
            }
            ++cell;
        }
-       os << "\\\\" << endl;
+       os << "\\\\\n";
        ret += TeXBottomHLine(os, i);
        bret = ret;
        if (IsLongTabular()) {
@@ -2229,29 +2249,121 @@ int LyXTabular::Latex(Buffer const * buf,
 }
 
 
-static void print_n_chars(ostream & os, unsigned char ch, int const n)
+int LyXTabular::DocBook(Buffer const * buf, ostream & os) const
+{
+    int ret = 0;
+
+    //+---------------------------------------------------------------------
+    //+                      first the opening preamble                    +
+    //+---------------------------------------------------------------------
+
+    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) {
+       case LYX_ALIGN_LEFT:
+           os << "left";
+           break;
+       case LYX_ALIGN_RIGHT:
+           os << "right";
+           break;
+       default:
+           os << "center";
+           break;
+       }
+       os << "\"/>\n";
+       ++ret;
+    }
+
+    //+---------------------------------------------------------------------
+    //+                      the single row and columns (cells)            +
+    //+---------------------------------------------------------------------
+
+    int cell = 0;
+    os << "<tbody>\n";
+    for (int i = 0; i < rows_; ++i) {
+        os << "<row>\n";
+       for (int j = 0; j < columns_; ++j) {
+           if (IsPartOfMultiColumn(i, j))
+               continue;
+           
+           os << "<entry align=\"";
+           switch (GetAlignment(cell)) {
+           case LYX_ALIGN_LEFT:
+               os << "left";
+               break;
+           case LYX_ALIGN_RIGHT:
+               os << "right";
+               break;
+           default:
+               os << "center";
+               break;
+           }
+           
+           os << "\" valign=\"";
+           switch (GetVAlignment(cell)) {
+           case LYX_VALIGN_TOP:
+               os << "top";
+               break;
+           case LYX_VALIGN_BOTTOM:
+               os << "bottom";
+               break;
+           case LYX_VALIGN_CENTER:
+               os << "middle";
+           }
+           os << "\"";
+           
+           if (IsMultiColumn(cell)) {
+               os << " namest=\"col" << j << "\" ";
+               os << "nameend=\"col" << j + cells_in_multicolumn(cell) - 1<< "\"";
+           }
+           
+           os << ">";
+           ret += GetCellInset(cell)->DocBook(buf, os);
+           os << "</entry>";
+           ++cell;
+       }
+       os << "</row>\n";
+    }
+    os << "</tbody>\n";
+    //+---------------------------------------------------------------------
+    //+                      the closing of the tabular                    +
+    //+---------------------------------------------------------------------
+
+    os << "</tgroup>";
+    ++ret;
+
+    return ret;
+}
+
+
+static
+inline
+void print_n_chars(ostream & os, unsigned char ch, int n)
 {
-    for(int i=0; i < n; ++i)
-       os << ch;
+       os << string(n, ch);
 }
 
+
 int LyXTabular::AsciiTopHLine(ostream & os, int row,
                              vector<unsigned int> const & clen) const
 {
     int const fcell = GetFirstCellInRow(row);
     int const n = NumberOfCellsInRow(fcell) + fcell;
-    int len;
-    int column = 0;
-    unsigned char ch;
     int tmp = 0;
 
     for (int i = fcell; i < n; ++i) {
-       if (TopLine(i))
+       if (TopLine(i)) {
            ++tmp;
+           break;
+       }
     }
     if (!tmp)
        return 0;
 
+    unsigned char ch;
     for (int i = fcell; i < n; ++i) {
        if (TopLine(i)) {
            if (LeftLine(i))
@@ -2263,8 +2375,8 @@ int LyXTabular::AsciiTopHLine(ostream & os, int row,
            os << "  ";
            ch = ' ';
        }
-       column = column_of_cell(i);
-       len = clen[column];
+       int column = column_of_cell(i);
+       int len = clen[column];
        while(IsPartOfMultiColumn(row, ++column))
            len += clen[column] + 4;
        print_n_chars(os, ch, len);
@@ -2287,18 +2399,18 @@ int LyXTabular::AsciiBottomHLine(ostream & os, int row,
 {
     int const fcell = GetFirstCellInRow(row);
     int const n = NumberOfCellsInRow(fcell) + fcell;
-    int len;
-    int column = 0;
-    unsigned char ch;
     int tmp = 0;
 
     for (int i = fcell; i < n; ++i) {
-       if (BottomLine(i))
+       if (BottomLine(i)) {
            ++tmp;
+           break;
+       }
     }
     if (!tmp)
        return 0;
 
+    unsigned char ch;
     for (int i = fcell; i < n; ++i) {
        if (BottomLine(i)) {
            if (LeftLine(i))
@@ -2310,8 +2422,8 @@ int LyXTabular::AsciiBottomHLine(ostream & os, int row,
            os << "  ";
            ch = ' ';
        }
-       column = column_of_cell(i);
-       len = clen[column];
+       int column = column_of_cell(i);
+       int len = clen[column];
        while(IsPartOfMultiColumn(row, ++column))
            len += clen[column] + 4;
        print_n_chars(os, ch, len);
@@ -2347,7 +2459,7 @@ int LyXTabular::AsciiPrintCell(Buffer const * buf, ostream & os,
        len2 += clen[column] + 4;
     len2 -= len1;
 
-    switch(GetAlignment(cell)) {
+    switch (GetAlignment(cell)) {
     default:
     case LYX_ALIGN_LEFT:
        len1 = 0;
@@ -2362,18 +2474,17 @@ int LyXTabular::AsciiPrintCell(Buffer const * buf, ostream & os,
        break;
     }
 
-    unsigned int i;
-    for(i=0; i < len1; ++i)
+    for (unsigned int i = 0; i < len1; ++i)
        os << " ";
     os << sstr.str();
-    for(i=0; i < len2; ++i)
+    for (unsigned int i = 0; i < len2; ++i)
        os << " ";
     if (RightLine(cell))
        os << " |";
     else
        os << "  ";
 
-    return ret * 0;
+    return ret * 0; // eh? (Lgb)
 }
 
 
@@ -2385,41 +2496,40 @@ int LyXTabular::Ascii(Buffer const * buf, ostream & os) const
     //+           first calculate the width of the single columns          +
     //+---------------------------------------------------------------------
     vector<unsigned int> clen(columns_);
-    int cell;
 
     // first all non (real) multicolumn cells!
-    for(int j = 0; j < columns_; ++j) {
+    for (int j = 0; j < columns_; ++j) {
        clen[j] = 0;
-       for(int i = 0; i < rows_; ++i) {
-           cell = GetCellNumber(i, j);
+       for (int i = 0; i < rows_; ++i) {
+           int cell = GetCellNumber(i, j);
            if (IsMultiColumn(cell, true))
                continue;
            ostringstream sstr;
-           (void)GetCellInset(cell)->Ascii(buf, sstr, 0);
+           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) {
-           cell = GetCellNumber(i, j);
+    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;
-           (void)GetCellInset(cell)->Ascii(buf, sstr, 0);
-           int len = (int)sstr.str().length();
-           int n = cells_in_multicolumn(cell);
-           for (int k = j; (len > 0) && (k < (j+n-1)); ++k)
+           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;
+           if (len > int(clen[j + n - 1]))
+               clen[j + n - 1] = len;
        }
     }
-    cell = 0;
-    for(int i = 0; i < rows_; ++i) {
+    int cell = 0;
+    for (int i = 0; i < rows_; ++i) {
        AsciiTopHLine(os, i, clen);
-       for(int j = 0; j < columns_; ++j) {
+       for (int j = 0; j < columns_; ++j) {
            if (IsPartOfMultiColumn(i,j))
                continue;
            ret += AsciiPrintCell(buf, os, cell, i, j, clen);
@@ -2428,6 +2538,7 @@ int LyXTabular::Ascii(Buffer const * buf, ostream & os) const
        os << endl;
        AsciiBottomHLine(os, i, clen);
     }
+    return ret;
 }
 
 
@@ -2437,13 +2548,19 @@ InsetText * LyXTabular::GetCellInset(int cell) const
 }
 
 
+InsetText * LyXTabular::GetCellInset(int row, int column) const
+{
+    return GetCellInset(GetCellNumber(row, column));
+}
+
+
 void LyXTabular::Validate(LaTeXFeatures & features) const
 {
     if (IsLongTabular())
         features.longtable = true;
     if (NeedRotating())
        features.rotating = true;
-    for(int cell = 0; cell < numberofcells; ++cell) {
+    for (int cell = 0; cell < numberofcells; ++cell) {
        if (GetVAlignment(cell) != LYX_VALIGN_TOP)
            features.array = true;
        GetCellInset(cell)->Validate(features);
@@ -2455,8 +2572,8 @@ LyXTabular::BoxType LyXTabular::UseParbox(int cell) const
 {
     LyXParagraph * par = GetCellInset(cell)->par;
 
-    for(; par; par = par->next) {
-       for(int i = 0; i < par->Last(); ++i) {
+    for (; par; par = par->next) {
+       for (int i = 0; i < par->Last(); ++i) {
            if (par->GetChar(i) == LyXParagraph::META_NEWLINE)
                return BOX_PARBOX;
        }