From 6eeafe15d0ce0cbb195f8c2ba15311192462f04c Mon Sep 17 00:00:00 2001
From: nyamatongwe <devnull@localhost>
Date: Tue, 12 Jun 2007 03:15:00 +0000
Subject: Fixed bug where setting the same range twice to the same value
 resulted in rest of document being set.

---
 src/RunStyles.cxx | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

(limited to 'src')

diff --git a/src/RunStyles.cxx b/src/RunStyles.cxx
index 76174fea4..13d48244b 100644
--- a/src/RunStyles.cxx
+++ b/src/RunStyles.cxx
@@ -138,14 +138,16 @@ bool RunStyles::FillRange(int &position, int value, int &fillLength) {
 			runEnd++;
 		}
 	}
-	styles->SetValueAt(runStart, value);
-	// Remove each old run over the range
-	for (int run=runStart+1; run<runEnd; run++) {
-		RemoveRun(runStart+1);
+	if (runStart < runEnd) {
+		styles->SetValueAt(runStart, value);
+		// Remove each old run over the range
+		for (int run=runStart+1; run<runEnd; run++) {
+			RemoveRun(runStart+1);
+		}
+		runEnd = RunFromPosition(end);
+		RemoveRunIfSameAsPrevious(runEnd);
+		RemoveRunIfSameAsPrevious(runStart);
 	}
-	runEnd = RunFromPosition(end);
-	RemoveRunIfSameAsPrevious(runEnd);
-	RemoveRunIfSameAsPrevious(runStart);
 	return true;
 }
 
-- 
cgit v1.2.3