Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial support for wide ncurses(multibyte output) #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shtirlic
Copy link
Contributor

  • vsave update to row scan, support wide chars
  • vrestore update to row restore, supports wide chars

Basic ncurses ouput(scrolling) should be smoother since now we are restoring and filling buffers by row instead of one by one char

@shtirlic shtirlic force-pushed the widecurses branch 2 times, most recently from 56f5d18 to 13b0056 Compare May 24, 2024 07:14
@vitaliy-aksyonov vitaliy-aksyonov marked this pull request as ready for review May 31, 2024 15:21
CMakeLists.txt Outdated Show resolved Hide resolved
goldlib/gcui/gvidbase.cpp Outdated Show resolved Hide resolved
goldlib/gcui/gvidbase.cpp Outdated Show resolved Hide resolved
goldlib/gcui/gvidbase.cpp Outdated Show resolved Hide resolved
goldlib/gcui/gvidbase.cpp Outdated Show resolved Hide resolved
goldlib/gcui/gvidbase.cpp Outdated Show resolved Hide resolved
@shtirlic shtirlic force-pushed the widecurses branch 3 times, most recently from d5231c6 to 817b907 Compare June 2, 2024 15:58
@spanevin
Copy link
Contributor

spanevin commented Jun 3, 2024

What about codacy warnings? Is there a way to move variables declaration under the same conditions as their usage?

@vitaliy-aksyonov
Copy link
Contributor

What about codacy warnings? Is there a way to move variables declaration under the same conditions as their usage?

I'd just move it under #ifdef

* vsave update to row scan, support wide chars
* vrestore update to row restore, supports wide chars

Basic ncurses ouput(scrolling) should be smoother since now we are
restoring and filling buffers by row instead of one by one char
@shtirlic
Copy link
Contributor Author

shtirlic commented Jun 4, 2024

Updated for codacy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants