]> git.lyx.org Git - lyx.git/blobdiff - src/changes.C
Scons: update_po target, part one: language_l10n.pot
[lyx.git] / src / changes.C
index 133c5b76aaf77b7f5897062fdef22e0d89b41ce7..46bfe48e54545694cc6dba1fc425fcc984921fc6 100644 (file)
@@ -30,27 +30,40 @@ using std::max;
  * Class Change has a changetime field that specifies the exact time at which
  * a specific change was made. The change time is used as a guidance for the
  * user while editing his document. Presently, it is not considered for LaTeX
- * export. To avoid that every keystroke results in a separate change, a 
- * tolerance interval of 5 minutes is used. That means if there are two adjacent
- * changes that only differ in their change time with abs(ct1 - ct2) < 300 sec,
- * they will be merged (and the later change time is preserved).
- * Technically, the check for equality (or similarity) is made in operator==(...).
- * The merging of similar changes happens in method merge().
+ * export.
+ * When merging two adjacent changes, the changetime is not considered,
+ * only the equality of the change type and author is checked (in method
+ * isSimilarTo(...)). If two changes are in fact merged (in method merge()),
+ * the later change time is preserved. 
  */
 
+bool Change::isSimilarTo(Change const & change)
+{
+       if (type != change.type) {
+               return false;
+       }
+
+       if (type == Change::UNCHANGED) {
+               return true;
+       }
+
+       return author == change.author;
+}
+
+
 bool operator==(Change const & l, Change const & r)
 {
        if (l.type != r.type) {
                return false;
        }
 
+       // two changes of type UNCHANGED are always equal
        if (l.type == Change::UNCHANGED) {
                return true;
        }
-
-       return l.author == r.author
-              // both changes made within 5 minutes?
-              && abs(difftime(l.changetime, r.changetime)) < 300;
+       
+       return l.author == r.author &&
+              l.changetime == r.changetime;
 }
 
 
@@ -72,18 +85,6 @@ bool operator!=(Changes::Range const & r1, Changes::Range const & r2)
 }
 
 
-bool Changes::Range::contains(Range const & r) const
-{
-       return r.start >= start && r.end <= end;
-}
-
-
-bool Changes::Range::contains(pos_type const pos) const
-{
-       return pos >= start && pos < end;
-}
-
-
 bool Changes::Range::intersects(Range const & r) const
 {
        return r.start < end && r.end > start; // end itself is not in the range!
@@ -98,7 +99,7 @@ void Changes::set(Change const & change, pos_type const pos)
 
 void Changes::set(Change const & change, pos_type const start, pos_type const end)
 {
-       if (lyxerr.debugging(Debug::CHANGES)) {
+       if (change.type != Change::UNCHANGED && lyxerr.debugging(Debug::CHANGES)) {
                lyxerr[Debug::CHANGES] << "setting change (type: " << change.type
                        << ", author: " << change.author << ", time: " << change.changetime
                        << ") in range (" << start << ", " << end << ")" << endl;
@@ -201,7 +202,7 @@ void Changes::erase(pos_type const pos)
 
 void Changes::insert(Change const & change, lyx::pos_type pos)
 {
-       if (lyxerr.debugging(Debug::CHANGES)) {
+       if (change.type != Change::UNCHANGED && lyxerr.debugging(Debug::CHANGES)) {
                lyxerr[Debug::CHANGES] << "Inserting change of type " << change.type
                        << " at position " << pos << endl;
        }
@@ -225,11 +226,10 @@ void Changes::insert(Change const & change, lyx::pos_type pos)
 }
 
 
-Change const Changes::lookup(pos_type const pos) const
+Change const Changes::lookup(pos_type const pos) const
 {
-       if (table_.empty()) {
-               return Change(Change::UNCHANGED);
-       }
+       static Change const noChange = Change(Change::UNCHANGED);
+               
        ChangeTable::const_iterator it = table_.begin();
        ChangeTable::const_iterator const end = table_.end();
 
@@ -238,7 +238,7 @@ Change const Changes::lookup(pos_type const pos) const
                        return it->change;
        }
 
-       return Change(Change::UNCHANGED);
+       return noChange;
 }
 
 
@@ -264,22 +264,18 @@ bool Changes::isChanged(pos_type const start, pos_type const end) const
 
 void Changes::merge()
 {
-       if (lyxerr.debugging(Debug::CHANGES)) {
-               lyxerr[Debug::CHANGES] << "merging changes..." << endl;
-       }
-
        ChangeTable::iterator it = table_.begin();
 
        while (it != table_.end()) {
                if (lyxerr.debugging(Debug::CHANGES)) {
-                       lyxerr[Debug::CHANGES] << "  found change of type " << it->change.type
+                       lyxerr[Debug::CHANGES] << "found change of type " << it->change.type
                                << " and range (" << it->range.start << ", " << it->range.end
                                << ")" << endl;
                }
 
                if (it->range.start == it->range.end) {
                        if (lyxerr.debugging(Debug::CHANGES)) {
-                               lyxerr[Debug::CHANGES] << "  removing empty range for pos "
+                               lyxerr[Debug::CHANGES] << "removing empty range for pos "
                                        << it->range.start << endl;
                        }
 
@@ -292,9 +288,9 @@ void Changes::merge()
                if (it + 1 == table_.end())
                        break;
 
-               if (it->change == (it + 1)->change && it->range.end == (it + 1)->range.start) {
+               if (it->change.isSimilarTo((it + 1)->change) && it->range.end == (it + 1)->range.start) {
                        if (lyxerr.debugging(Debug::CHANGES)) {
-                               lyxerr[Debug::CHANGES] << "  merging ranges (" << it->range.start << ", "
+                               lyxerr[Debug::CHANGES] << "merging ranges (" << it->range.start << ", "
                                        << it->range.end << ") and (" << (it + 1)->range.start << ", "
                                        << (it + 1)->range.end << ")" << endl;
                        }