diff options
author | Neil <nyamatongwe@gmail.com> | 2021-07-17 10:04:20 +1000 |
---|---|---|
committer | Neil <nyamatongwe@gmail.com> | 2021-07-17 10:04:20 +1000 |
commit | bea9d7216879bc80a8513d3374a931cba14f6951 (patch) | |
tree | 0a34eab53e9e7e8c3b91fec994f64b1bfba42df9 /src | |
parent | 046b280114d11ff24b179dbe4df44127e80d39b5 (diff) | |
download | scintilla-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.cxx | 14 |
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)); } |