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.

Sign up for GitHub

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

BSキーでカーソル前を削除時に画面が余計に再描画される事を防ぐ #1754

Merged
merged 1 commit into from
Jan 14, 2022
Merged

BSキーでカーソル前を削除時に画面が余計に再描画される事を防ぐ #1754

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions sakura_core/cmd/CViewCommander_Edit.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -857,6 +857,7 @@ void CViewCommander::Command_DELETE_BACK( void )
GetCaret().GetCaretLogicPos()
)
);
GetDocument()->m_cDocEditor.m_nOpeBlkRedawCount++;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正内容はここだけ（1行追加）ですよね？
　↓
他の処理から「本来必要な処理」を抜いてきて貼り付けた、ってことですよね？

【このPRの修正の成り立ち（推定）】
再描画が遅くならないようにする機構がもともと存在していた。
BSキー押下時の処理にはこの機構を有効にするための1文が抜けている（ように見えた）。
・・・ので、「～したときのコード」から必要な1文をコピってきた。

PR本文の説明とまるで違うので、そうではないんだろうな、と思いました。
成り立ちが上記想定に一致するなら入れてしまってもいいと思います。

現状、自分はこの件で困ってないです。
BSだとエディタによっては処理遅延が発生するので、
削除範囲を選択してから削除するクセがついてるためかと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

既存の記述を参考というかコピってきたのはその通りです。DELキーだとビューの全再描画はされていなかったのでその仕組みを真似ています。

PR本文の説明にそれが書かれていないのは、多分書いた人に親切心が足りてないからだと思われます。

なおミニマップを表示していない時でもBSキー時に全ビューを再描画する挙動は同じなのですがそこまで重くないです。気づかない間に密かに無駄遣いしていたんですね。

}
m_pCommanderView->DeleteData( true );
}
Expand Down