Fix bug where too many characters may be deleted when a rectangular selection is

deleted.

4 files changed, 22 insertions, 2 deletions

diff --git a/doc/ScintillaHistory.html b/doc/ScintillaHistory.html
index 3bddd2d26..77fae0685 100644
--- a/doc/ScintillaHistory.html
+++ b/doc/ScintillaHistory.html
@@ -460,6 +460,17 @@
       </li>
     </ul>
     <h3>
+       <a href="http://prdownloads.sourceforge.net/scintilla/scite345.zip?download">Release 3.4.5</a>
+    </h3>
+    <ul>
+	<li>
+	Released 3 July 2014.
+	</li>
+	<li>
+	Bug fixed where style data was stale when deleting a rectangular selection.
+	</li>
+    <ul>
+    <h3>
        <a href="http://prdownloads.sourceforge.net/scintilla/scite344.zip?download">Release 3.4.4</a>
     </h3>
     <ul>
diff --git a/src/Editor.cxx b/src/Editor.cxx
index db0aecf43..d3e5f3090 100644
--- a/src/Editor.cxx
+++ b/src/Editor.cxx
@@ -4321,6 +4321,7 @@ void Editor::DelChar() {
 }
 
 void Editor::DelCharBack(bool allowLineStartDeletion) {
+	RefreshStyleData();
 	if (!sel.IsRectangular())
 		FilterSelections();
 	if (sel.IsRectangular())
diff --git a/src/Selection.cxx b/src/Selection.cxx
index b46dca890..52ed5774e 100644
--- a/src/Selection.cxx
+++ b/src/Selection.cxx
@@ -81,6 +81,11 @@ int SelectionRange::Length() const {
 	}
 }
 
+void SelectionRange::MoveForInsertDelete(bool insertion, int startChange, int length) {
+	caret.MoveForInsertDelete(insertion, startChange, length);
+	anchor.MoveForInsertDelete(insertion, startChange, length);
+}
+
 bool SelectionRange::Contains(int pos) const {
 	if (anchor > caret)
 		return (pos >= caret.Position()) && (pos <= anchor.Position());
@@ -283,9 +288,11 @@ int Selection::Length() const {
 
 void Selection::MovePositions(bool insertion, int startChange, int length) {
 	for (size_t i=0; i<ranges.size(); i++) {
-		ranges[i].caret.MoveForInsertDelete(insertion, startChange, length);
-		ranges[i].anchor.MoveForInsertDelete(insertion, startChange, length);
+		ranges[i].MoveForInsertDelete(insertion, startChange, length);
 	}
+	if (selType == selRectangle) {
+		rangeRectangular.MoveForInsertDelete(insertion, startChange, length);
+	} 
 }
 
 void Selection::TrimSelection(SelectionRange range) {
diff --git a/src/Selection.h b/src/Selection.h
index 9e95fb043..499f83771 100644
--- a/src/Selection.h
+++ b/src/Selection.h
@@ -115,6 +115,7 @@ struct SelectionRange {
 		anchor.SetVirtualSpace(0);
 		caret.SetVirtualSpace(0);
 	}
+	void MoveForInsertDelete(bool insertion, int startChange, int length);
 	bool Contains(int pos) const;
 	bool Contains(SelectionPosition sp) const;
 	bool ContainsCharacter(int posCharacter) const;