Merge remote branch 'origin/master' into f14/master

This commit is contained in:
Rex Dieter 2010-10-02 15:29:52 -05:00
commit 8493938a76
6 changed files with 11 additions and 348 deletions

2
.gitignore vendored
View File

@ -1 +1 @@
kdelibs-4.5.1.tar.bz2
/kdelibs-4.5.2.tar.bz2

View File

@ -11,8 +11,8 @@
#define apidocs_qch 1
Summary: KDE Libraries
Version: 4.5.1
Release: 4%{?dist}
Version: 4.5.2
Release: 2%{?dist}
Name: kdelibs
Epoch: 6
@ -82,10 +82,6 @@ Patch24: kdelibs-4.3.1-drkonq.patch
# paths (like /usr/lib64) already! With this, we can drop
# -DCMAKE_SKIP_RPATH:BOOL=ON (finally)
Patch27: kdelibs-4.4.80-no_rpath.patch
# patches backported, fixed in coming 4.5.2, fix crashing kdevelop
Patch30: kdelibs_4.5.1_fix_kdevelop-crash_katetextblock_cpp.patch
Patch31: kdelibs_4.5.1_fix_kdevelop-crash_katetexthistory_cpp.patch
Patch32: kdelibs_4.5.1_fix_kdevelop-crash_katetexthistory_h.patch
## upstreamable
# add gpg2 support to knewstuff, rough first try s/gpg/gpg2/
@ -93,7 +89,7 @@ Patch32: kdelibs_4.5.1_fix_kdevelop-crash_katetexthistory_h.patch
# TODO: try to use either gpg or gpg2, whichever is available
Patch50: kdelibs-4.5.1-knewstuff_gpg2.patch
## 4.4 upstream
## 4.5 upstream
## trunk upstream
@ -260,9 +256,6 @@ format for use with the Qt 4 Assistant or KDevelop 4.
%patch50 -p1 -b .knewstuff_gpg2
# upstream patches
%patch30 -p0
%patch31 -p0
%patch32 -p0
# security fix
%patch200 -p1 -b .CVE-2009-2702
@ -510,6 +503,12 @@ rm -rf %{buildroot}
%changelog
* Fri Oct 01 2010 Rex Dieter <rdieter@fedoraproject.org> 4.5.2-2
- rebuild (phonon)
* Fri Oct 01 2010 Rex Dieter <rdieter@fedoraproject.org> 4.5.2-1
- 4.5.2
* Fri Sep 10 2010 Thomas Janssen <thomasj@fedoraproject.org> 4.5.1-4
- backport patches to fix a crashing kdevelop (kde 4.5.1 only)

View File

@ -1,19 +0,0 @@
--- kate/buffer/katetextblock.cpp 2010/09/06 00:49:03 1171997
+++ kate/buffer/katetextblock.cpp 2010/09/06 00:49:36 1171998
@@ -355,6 +355,7 @@
// get text
QString &textOfLine = m_lines[line]->textReadWrite ();
+ int oldLength = textOfLine.size ();
// check if valid column
Q_ASSERT (range.start().column() >= 0);
@@ -371,7 +372,7 @@
/**
* notify the text history
*/
- m_buffer->history().removeText (range);
+ m_buffer->history().removeText (range, oldLength);
/**
* cursor and range handling below

View File

@ -1,289 +0,0 @@
--- kate/buffer/katetexthistory.cpp 2010/09/06 00:49:03 1171997
+++ kate/buffer/katetexthistory.cpp 2010/09/06 00:49:36 1171998
@@ -94,7 +94,7 @@
addEntry (entry);
}
-void TextHistory::removeText (const KTextEditor::Range &range)
+void TextHistory::removeText (const KTextEditor::Range &range, int oldLineLength)
{
// create and add new entry
Entry entry;
@@ -102,6 +102,7 @@
entry.line = range.start().line ();
entry.column = range.start().column ();
entry.length = range.end().column() - range.start().column();
+ entry.oldLineLength = oldLineLength;
addEntry (entry);
}
@@ -315,11 +316,131 @@
}
}
+void TextHistory::Entry::reverseTransformCursor (int &cursorLine, int &cursorColumn, bool moveOnInsert) const
+{
+ /**
+ * handle all history types
+ */
+ switch (type) {
+ /**
+ * Wrap a line
+ */
+ case WrapLine:
+ /**
+ * ignore this line
+ */
+ if (cursorLine <= line)
+ return;
+
+ /**
+ * next line is unwrapped
+ */
+ if (cursorLine == line + 1) {
+ /**
+ * adjust column
+ */
+ cursorColumn = cursorColumn + column;
+ }
+
+ /**
+ * always decrement cursor line
+ */
+ cursorLine -= 1;
+ return;
+
+ /**
+ * Unwrap a line
+ */
+ case UnwrapLine:
+ /**
+ * ignore lines before unwrapped one
+ */
+ if (cursorLine < line - 1)
+ return;
+
+ /**
+ * we unwrap this line, try to adjust cursor column if needed
+ */
+ if (cursorLine == line - 1) {
+ /**
+ * skip cursors with to small columns
+ */
+ if (cursorColumn <= oldLineLength) {
+ if (cursorColumn < oldLineLength || !moveOnInsert)
+ return;
+ }
+
+ cursorColumn -= oldLineLength;
+ }
+
+ /**
+ * increase cursor line
+ */
+ cursorLine += 1;
+ return;
+
+ /**
+ * Insert text
+ */
+ case InsertText:
+ /**
+ * only interesting, if same line
+ */
+ if (cursorLine != line)
+ return;
+
+ // skip cursors with too small column
+ if (cursorColumn <= column)
+ return;
+
+ // patch column of cursor
+ if (cursorColumn - length < column)
+ cursorColumn = column;
+ else
+ cursorColumn -= length;
+
+ return;
+
+ /**
+ * Remove text
+ */
+ case RemoveText:
+ /**
+ * only interesting, if same line
+ */
+ if (cursorLine != line)
+ return;
+
+ // skip cursors with too small column
+ if (cursorColumn <= column)
+ if (cursorColumn < column || !moveOnInsert)
+ return;
+
+ // patch column of cursor
+ if (cursorColumn <= oldLineLength)
+ cursorColumn += length;
+
+ // special handling if cursor behind the real line, e.g. non-wrapping cursor in block selection mode
+ else if (cursorColumn < oldLineLength + length)
+ cursorColumn = oldLineLength + length;
+ return;
+
+ /**
+ * nothing
+ */
+ default:
+ return;
+ }
+}
+
void TextHistory::transformCursor (int& line, int& column, KTextEditor::MovingCursor::InsertBehavior insertBehavior, qint64 fromRevision, qint64 toRevision)
{
/**
- * -1 special meaning for toRevision
+ * -1 special meaning for from/toRevision
*/
+ if (fromRevision == -1)
+ fromRevision = revision ();
+
if (toRevision == -1)
toRevision = revision ();
@@ -333,7 +454,7 @@
* some invariants must hold
*/
Q_ASSERT (!m_historyEntries.empty ());
- Q_ASSERT (fromRevision < toRevision);
+ Q_ASSERT (fromRevision != toRevision);
Q_ASSERT (fromRevision >= m_firstHistoryEntryRevision);
Q_ASSERT (fromRevision < (m_firstHistoryEntryRevision + m_historyEntries.size()));
Q_ASSERT (toRevision >= m_firstHistoryEntryRevision);
@@ -343,9 +464,20 @@
* transform cursor
*/
bool moveOnInsert = insertBehavior == KTextEditor::MovingCursor::MoveOnInsert;
- for (int rev = fromRevision - m_firstHistoryEntryRevision + 1; rev <= (toRevision - m_firstHistoryEntryRevision); ++rev) {
- const Entry &entry = m_historyEntries[rev];
- entry.transformCursor (line, column, moveOnInsert);
+
+ /**
+ * forward or reverse transform?
+ */
+ if (toRevision > fromRevision) {
+ for (int rev = fromRevision - m_firstHistoryEntryRevision + 1; rev <= (toRevision - m_firstHistoryEntryRevision); ++rev) {
+ const Entry &entry = m_historyEntries[rev];
+ entry.transformCursor (line, column, moveOnInsert);
+ }
+ } else {
+ for (int rev = fromRevision - m_firstHistoryEntryRevision; rev >= (toRevision - m_firstHistoryEntryRevision + 1); --rev) {
+ const Entry &entry = m_historyEntries[rev];
+ entry.reverseTransformCursor (line, column, moveOnInsert);
+ }
}
}
@@ -361,8 +493,11 @@
}
/**
- * -1 special meaning for toRevision
+ * -1 special meaning for from/toRevision
*/
+ if (fromRevision == -1)
+ fromRevision = revision ();
+
if (toRevision == -1)
toRevision = revision ();
@@ -376,40 +511,68 @@
* some invariants must hold
*/
Q_ASSERT (!m_historyEntries.empty ());
- Q_ASSERT (fromRevision < toRevision);
+ Q_ASSERT (fromRevision != toRevision);
Q_ASSERT (fromRevision >= m_firstHistoryEntryRevision);
Q_ASSERT (fromRevision < (m_firstHistoryEntryRevision + m_historyEntries.size()));
Q_ASSERT (toRevision >= m_firstHistoryEntryRevision);
Q_ASSERT (toRevision < (m_firstHistoryEntryRevision + m_historyEntries.size()));
-
+
/**
* transform cursors
*/
-
+
// first: copy cursors, without range association
int startLine = range.start().line(), startColumn = range.start().column(), endLine = range.end().line(), endColumn = range.end().column();
bool moveOnInsertStart = !(insertBehaviors & KTextEditor::MovingRange::ExpandLeft);
bool moveOnInsertEnd = (insertBehaviors & KTextEditor::MovingRange::ExpandRight);
- for (int rev = fromRevision - m_firstHistoryEntryRevision + 1; rev <= (toRevision - m_firstHistoryEntryRevision); ++rev) {
- const Entry &entry = m_historyEntries[rev];
-
- entry.transformCursor (startLine, startColumn, moveOnInsertStart);
-
- entry.transformCursor (endLine, endColumn, moveOnInsertEnd);
-
- // got empty?
- if(endLine < startLine || (endLine == startLine && endColumn <= startColumn))
- {
- if (invalidateIfEmpty) {
- range = KTextEditor::Range::invalid();
- return;
- }
- else{
- // else normalize them
- endLine = startLine;
- endColumn = startColumn;
- }
+
+ /**
+ * forward or reverse transform?
+ */
+ if (toRevision > fromRevision) {
+ for (int rev = fromRevision - m_firstHistoryEntryRevision + 1; rev <= (toRevision - m_firstHistoryEntryRevision); ++rev) {
+ const Entry &entry = m_historyEntries[rev];
+
+ entry.transformCursor (startLine, startColumn, moveOnInsertStart);
+
+ entry.transformCursor (endLine, endColumn, moveOnInsertEnd);
+
+ // got empty?
+ if(endLine < startLine || (endLine == startLine && endColumn <= startColumn))
+ {
+ if (invalidateIfEmpty) {
+ range = KTextEditor::Range::invalid();
+ return;
+ }
+ else{
+ // else normalize them
+ endLine = startLine;
+ endColumn = startColumn;
+ }
+ }
+ }
+ } else {
+ for (int rev = fromRevision - m_firstHistoryEntryRevision ; rev >= (toRevision - m_firstHistoryEntryRevision + 1); --rev) {
+ const Entry &entry = m_historyEntries[rev];
+
+ entry.reverseTransformCursor (startLine, startColumn, moveOnInsertStart);
+
+ entry.reverseTransformCursor (endLine, endColumn, moveOnInsertEnd);
+
+ // got empty?
+ if(endLine < startLine || (endLine == startLine && endColumn <= startColumn))
+ {
+ if (invalidateIfEmpty) {
+ range = KTextEditor::Range::invalid();
+ return;
+ }
+ else{
+ // else normalize them
+ endLine = startLine;
+ endColumn = startColumn;
+ }
+ }
}
}

View File

@ -1,28 +0,0 @@
--- kate/buffer/katetexthistory.h 2010/09/06 00:49:03 1171997
+++ kate/buffer/katetexthistory.h 2010/09/06 00:49:36 1171998
@@ -99,6 +99,14 @@
* @param moveOnInsert behavior of this cursor on insert of text at it's position
*/
void transformCursor (int &line, int &column, bool moveOnInsert) const;
+
+ /**
+ * reverse transform cursor for this history entry
+ * @param line line number of the cursor to transform
+ * @param column column number of the cursor to transform
+ * @param moveOnInsert behavior of this cursor on insert of text at it's position
+ */
+ void reverseTransformCursor (int &line, int &column, bool moveOnInsert) const;
/**
* Types of entries, matching editing primitives of buffer and placeholder
@@ -195,8 +203,9 @@
/**
* Notify about remove text at given range.
* @param range range of text to remove, must be on one line only.
+ * @param oldLineLength text length of the line before this remove
*/
- void removeText (const KTextEditor::Range &range);
+ void removeText (const KTextEditor::Range &range, int oldLineLength);
/**
* Generic function to add a entry to the history. Is used by the above functions for the different editing primitives.

View File

@ -1 +1 @@
f91f16b4349f8a2ca41dab5ca95f3877 kdelibs-4.5.1.tar.bz2
be66ec28293f5bed92212a740ded2e5e kdelibs-4.5.2.tar.bz2