]> git.lyx.org Git - lyx.git/blobdiff - src/ParagraphMetrics.cpp
prepare Qt 5.6 builds
[lyx.git] / src / ParagraphMetrics.cpp
index bff6eb6d77babaae3cb5f6b23087025a8d1c59ff..ad217cfa604765859e3ac459646636173a4e44ee 100644 (file)
@@ -92,19 +92,22 @@ size_t ParagraphMetrics::computeRowSignature(Row const & row,
        for (pos_type i = row.pos(); i < row.endpos(); ++i) {
                char_type const b[] = { par_->getChar(i) };
                crc.process_bytes(b, sizeof(char_type));
-               if (bparams.trackChanges) {
+               if (bparams.track_changes) {
                        Change change = par_->lookupChange(i);
                        char_type const b[] = { static_cast<char_type>(change.type) };
                        // 1 byte is enough to encode Change::Type
                        crc.process_bytes(b, 1);
-               }                       
+               }
        }
 
        Dimension const & d = row.dimension();
        char_type const b[] = { static_cast<char_type>(row.sel_beg),
-                               static_cast<char_type>(row.sel_end),
-               row.begin_margin_sel, row.end_margin_sel, d.wid, d.asc, d.des};
+                               static_cast<char_type>(row.sel_end),
+                               row.begin_margin_sel,
+                               row.end_margin_sel,
+                               d.wid, d.asc, d.des };
        crc.process_bytes(b, sizeof(b));
+       crc.process_bytes(&row.separator, sizeof(row.separator));
 
        return crc.checksum();
 }
@@ -116,27 +119,9 @@ void ParagraphMetrics::setPosition(int position)
 }
 
 
-Dimension const & ParagraphMetrics::insetDimension(Inset const * inset) const
-{
-       InsetDims::const_iterator it = inset_dims_.find(inset);
-       if (it != inset_dims_.end())
-               return it->second;
-
-       static Dimension dummy;
-       return dummy;
-}
-
-
-void ParagraphMetrics::setInsetDimension(Inset const * inset,
-               Dimension const & dim)
-{
-       inset_dims_[inset] = dim;
-}
-
-
 Row & ParagraphMetrics::getRow(pos_type pos, bool boundary)
 {
-       LBUFERR(!rows().empty(), _("ParagraphMetrics has no rows!"));
+       LBUFERR(!rows().empty());
 
        // If boundary is set we should return the row on which
        // the character before is inside.
@@ -155,7 +140,7 @@ Row & ParagraphMetrics::getRow(pos_type pos, bool boundary)
 
 Row const & ParagraphMetrics::getRow(pos_type pos, bool boundary) const
 {
-       LBUFERR(!rows().empty(), _("ParagraphMetrics has no rows!"));
+       LBUFERR(!rows().empty());
 
        // If boundary is set we should return the row on which
        // the character before is inside.
@@ -174,7 +159,7 @@ Row const & ParagraphMetrics::getRow(pos_type pos, bool boundary) const
 
 size_t ParagraphMetrics::pos2row(pos_type pos) const
 {
-       LBUFERR(!rows().empty(), _("ParagraphMetrics has no rows!"));
+       LBUFERR(!rows().empty());
 
        RowList::const_iterator rit = rows_.end();
        RowList::const_iterator const begin = rows_.begin();
@@ -190,8 +175,7 @@ void ParagraphMetrics::dump() const
 {
        lyxerr << "Paragraph::dump: rows.size(): " << rows_.size() << endl;
        for (size_t i = 0; i != rows_.size(); ++i) {
-               lyxerr << "  row " << i << ":   ";
-               rows_[i].dump();
+               lyxerr << "  row " << i << ":   " << rows_[i];
        }
 }
 
@@ -210,37 +194,7 @@ int ParagraphMetrics::rightMargin(BufferView const & bv) const
 }
 
 
-int ParagraphMetrics::singleWidth(pos_type pos, Font const & font) const
-{
-       // The most special cases are handled first.
-       if (Inset const * inset = par_->getInset(pos))
-               return insetDimension(inset).wid;
-
-       char_type c = par_->getChar(pos);
-
-       if (c == '\t')
-               return 4 * theFontMetrics(font).width(' ');
-       
-       if (!isPrintable(c))
-               return theFontMetrics(font).width(c);
-
-       Language const * language = font.language();
-       if (language->rightToLeft()) {
-               if (language->lang() == "arabic_arabtex" ||
-                       language->lang() == "arabic_arabi" ||
-                       language->lang() == "farsi") {
-                               if (Encodings::isArabicComposeChar(c))
-                                       return 0;
-                               c = par_->transformChar(c, pos);
-               } else if (language->lang() == "hebrew" &&
-                               Encodings::isHebrewComposeChar(c)) {
-                       return 0;
-               }
-       }
-       return theFontMetrics(font).width(c);
-}
-
-
+// FIXME: this code seems bogus. Audit and rewrite (see bug #9860).
 bool ParagraphMetrics::hfillExpansion(Row const & row, pos_type pos) const
 {
        if (!par_->isHfill(pos))
@@ -269,7 +223,7 @@ bool ParagraphMetrics::hfillExpansion(Row const & row, pos_type pos) const
        // the specified position that is neither a newline nor an hfill,
        // the hfill will be expanded, otherwise it won't
        for (pos_type i = row.pos(); i < pos; i++) {
-               if (!par_->isNewline(i) && !par_->isHfill(i))
+               if (!par_->isNewline(i) && !par_->isEnvSeparator(i) && !par_->isHfill(i))
                        return true;
        }
        return false;