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

Fix display problem of pdcurses #336

Merged
merged 1 commit into from
Sep 26, 2018
Merged

Conversation

Hamayama
Copy link
Collaborator

PDCurses でしばしば画面が乱れる件ですが、やっと原因が分かったため対策しました。

PDCurses の getch が、内部で wrefresh を呼んでおり、
Lem の入力用スレッドと本体スレッドの両方から画面を更新していました。。。

対策として、charms/ll:newpad で作成したダミーの画面で getch を発行するようにしました。

(参考URL:
https://github.com/wmcbrine/PDCurses/blob/b7135bc906413898c13e47ec9e154ce3d642c939/pdcurses/getch.c#L193
https://stackoverflow.com/questions/19748685/curses-library-why-does-getch-clear-my-screen )

これでだいぶ表示が安定すると思います。

その他、少し修正しました。
( M-( と M-) が使えない件の対応。lem-if:redraw-view-after の処理見直し等)

また、本修正により、ConEmu の ansi color 設定 が ON でも、
(事前に chcp 65001 を実行しておけば) 正常に表示できるようになりました。

@cxxxr cxxxr requested a review from snmsts September 26, 2018 12:59
@snmsts
Copy link
Member

snmsts commented Sep 26, 2018

変更の意図の説明と対応する部分はなるほど…な感じ。

@snmsts snmsts merged commit 43b23a5 into lem-project:master Sep 26, 2018
@Hamayama Hamayama deleted the pdgetch branch December 21, 2018 14:05
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

2 participants