zhangyiss/FTXUI
1
0
Fork 0
mirror of https://github.com/ArthurSonzogni/FTXUI.git synced 2025-11-04 13:38:14 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Avoid making new allocation to clear the screen. (#420)

Browse Source 
Previously, a new 2D vector was allocated for every new frame. This
caused a lot of temporary allocation to be made.

This patch modify "Screen::Clear" so that it do make a new allocation,
but clear the existing one instead.

Bug:https://github.com/ArthurSonzogni/FTXUI/issues/290#issuecomment-1153327251
This commit is contained in:
Arthur Sonzogni
2022-06-13 21:49:36 +02:00
committed by GitHub
parent 925a7578d4
commit 81e086788d
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/ftxui/screen/screen.cpp
View File

@@ -487,8 +487,11 @@ std::string Screen::ResetPosition(bool clear) const {
/// @brief Clear all the pixel from the screen. /// @brief Clear all the pixel from the screen.
void Screen::Clear() { void Screen::Clear() {
pixels_ = std::vector<std::vector<Pixel>>(dimy_, for (auto& line : pixels_) {
std::vector<Pixel>(dimx_, Pixel())); for (auto& cell : line) {
cell = Pixel();
}
}
cursor_.x = dimx_ - 1; cursor_.x = dimx_ - 1;
cursor_.y = dimy_ - 1; cursor_.y = dimy_ - 1;
} }