Backed out changeset: 7caa35787c19

Change made reverse iteration dangerous.

1 files changed, 5 insertions, 4 deletions

diff --git a/src/Document.cxx b/src/Document.cxx
index c25dc84b9..7d34dace1 100644
--- a/src/Document.cxx
+++ b/src/Document.cxx
@@ -767,23 +767,24 @@ bool Document::NextCharacter(int &pos, int moveDir) const {
 	}
 }
 
+// Return -1  on out-of-bounds
 int SCI_METHOD Document::GetRelativePosition(int positionStart, int characterOffset) const {
 	int pos = positionStart;
 	if (dbcsCodePage) {
 		const int increment = (characterOffset > 0) ? 1 : -1;
 		while (characterOffset != 0) {
 			const int posNext = NextPosition(pos, increment);
-			if (posNext == Length())
-				return posNext;
-			else if (posNext == pos)
+			if (posNext == pos)
 				return INVALID_POSITION;
 			pos = posNext;
 			characterOffset -= increment;
 		}
 	} else {
 		pos = positionStart + characterOffset;
+		if ((pos < 0) || (pos > Length()))
+			return INVALID_POSITION;
 	}
-	return ClampPositionIntoDocument(pos);
+	return pos;
 }
 
 int Document::GetRelativePositionUTF16(int positionStart, int characterOffset) const {