xenia/kakoune
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

buffer end should not be in a selection

Browse Source
This commit is contained in:
Maxime Coste 2013-05-23 19:38:51 +02:00
parent 462da52218
commit 9a80a58ff4
3 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/selection.cc
View File

@ -32,8 +32,8 @@ String Range::content() const
void Range::check_invariant() const
{
#ifdef KAK_DEBUG
kak_assert(m_first.is_valid());
kak_assert(m_last.is_valid());
kak_assert(m_first.is_valid() and not m_first.is_end());
kak_assert(m_last.is_valid() and not m_last.is_end());
kak_assert(utf8::is_character_start(m_first));
kak_assert(utf8::is_character_start(m_last));
#endif

2
src/selectors.cc
View File

@ -504,6 +504,8 @@ Selection select_whole_lines(const Selection& selection)
++to_line_start;
skip_while(to_line_end, [](char cur) { return not is_eol(cur); });
if (is_end(to_line_end))
--to_line_end;
return Selection(first, last);
}

2
src/unit_tests.cc
View File

@ -67,7 +67,7 @@ void test_editor()
}
editor.undo();
Selection sel{ buffer.iterator_at_line_begin(2_line), buffer.end() };
Selection sel{ buffer.iterator_at_line_begin(2_line), buffer.end()-1 };
editor.select(sel, SelectMode::Replace);
editor.insert("",InsertMode::Replace);
kak_assert(not editor.main_selection().first().is_end());