aboutsummaryrefslogtreecommitdiffhomepage
path: root/src
diff options
context:
space:
mode:
authorNeil <nyamatongwe@gmail.com>2021-07-17 10:04:20 +1000
committerNeil <nyamatongwe@gmail.com>2021-07-17 10:04:20 +1000
commitbea9d7216879bc80a8513d3374a931cba14f6951 (patch)
tree0a34eab53e9e7e8c3b91fec994f64b1bfba42df9 /src
parent046b280114d11ff24b179dbe4df44127e80d39b5 (diff)
downloadscintilla-mirror-bea9d7216879bc80a8513d3374a931cba14f6951.tar.gz
Fix minor inconsistency with word searching results at start and end of document
which were considered word start / end positions even when first or last character did not fit requirements.
Diffstat (limited to 'src')
-rw-r--r--src/Document.cxx14
1 files changed, 9 insertions, 5 deletions
diff --git a/src/Document.cxx b/src/Document.cxx
index 3a3627fdc..b5ec9a275 100644
--- a/src/Document.cxx
+++ b/src/Document.cxx
@@ -1923,23 +1923,27 @@ constexpr bool IsWordEdge(CharacterClass cc, CharacterClass ccNext) noexcept {
bool Document::IsWordStartAt(Sci::Position pos) const {
if (pos >= LengthNoExcept())
return false;
- if (pos > 0) {
+ if (pos >= 0) {
const CharacterExtracted cePos = CharacterAfter(pos);
- const CharacterExtracted cePrev = CharacterBefore(pos);
+ // At start of document, treat as if space before so can be word start
+ const CharacterExtracted cePrev = (pos > 0) ?
+ CharacterBefore(pos) : CharacterExtracted(' ', 1);
return IsWordEdge(WordCharacterClass(cePos.character), WordCharacterClass(cePrev.character));
}
return true;
}
/**
- * Check that the character at the given position is a word or punctuation character and that
+ * Check that the character before the given position is a word or punctuation character and that
* the next character is of a different character class.
*/
bool Document::IsWordEndAt(Sci::Position pos) const {
if (pos <= 0)
return false;
- if (pos < LengthNoExcept()) {
- const CharacterExtracted cePos = CharacterAfter(pos);
+ if (pos <= LengthNoExcept()) {
+ // At end of document, treat as if space after so can be word end
+ const CharacterExtracted cePos = (pos < LengthNoExcept()) ?
+ CharacterAfter(pos) : CharacterExtracted(' ', 1);
const CharacterExtracted cePrev = CharacterBefore(pos);
return IsWordEdge(WordCharacterClass(cePrev.character), WordCharacterClass(cePos.character));
}