Jasima/godot-xterm
1
0
Fork 0
mirror of https://github.com/lihop/godot-xterm.git synced 2024-11-21 17:20:25 +01:00
Code Issues Projects Releases Packages Wiki Activity

fix(term): copy_selection() when nothing selected

Browse source 
Fixes the copy_selection() method so that it returns and empty string
when called and nothing is selected. Previously printed an error.
This commit is contained in:
Leroy Hopson 2024-04-28 15:25:26 +12:00 committed by Leroy Hopson
parent 3ca272c615
commit 43303a51bf
2 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
addons/godot_xterm/native/src/terminal.cpp
View file

@ -648,9 +648,11 @@ String Terminal::_copy_screen(ScreenCopyFunction func) {
char *out;
PackedByteArray data;
data.resize(func(screen, &out));
memcpy(data.ptrw(), out, data.size());
std::free(out);
data.resize(std::max(func(screen, &out), 0));
if (data.size() > 0) {
memcpy(data.ptrw(), out, data.size());
std::free(out);
}
return data.get_string_from_utf8();
}

3
test/test_terminal.gd
View file

@ -222,6 +222,9 @@ class TestCopy:
subject.write(text)
assert_string_contains(subject.copy_all(), text)
func test_copy_selection_when_nothing_selected():
assert_eq(subject.copy_selection(), "")
class TestClear:
extends TerminalTest