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

縦スクロール時に不必要にルーラーの再描画がされないように対策 #660

Merged
merged 1 commit into from
Dec 2, 2018
Merged

縦スクロール時に不必要にルーラーの再描画がされないように対策 #660

merged 1 commit into from
Dec 2, 2018

Conversation

beru
Copy link
Contributor

@beru beru commented Dec 1, 2018

スクロール行数が表示行数を超える場合にクライアント領域全体が再描画の対象領域にされていましたが、それを限定してルーラー部分の再描画がされないように対策しました。

スクロールバーのサム(つまみ)をドラッグ操作で縦スクロールする際のCPU負荷を下げるのが目的です。自分の環境だとPerformance Profiler で起動後に何かテキストファイル(例えば C:\Windows\PolicyDefinitions\inetres.admx) を開いて縦スクロールバーをひたすら上下に動かす操作をした場合の CRuler::DispRuler のCPU使用率が、3% 近くあったのが 0.25% 辺りに低下します。

@beru beru added the 🚅 speed up 🚀 高速化 label Dec 1, 2018
@beru beru changed the title スクロール時に不必要にルーラーの再描画がされないように対策 縦スクロール時に不必要にルーラーの再描画がされないように対策 Dec 1, 2018
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

LGTMです。

その他関連個所をすべて洗い出して対応するのは大変なので、
確実に影響を与えていると分かっている箇所を先行して対応する、というのはアリだと思います。

この修正で影響するのは縦スクロールだけなので、一応ぐりぐりしてみました。
想定外の変な挙動もしてないようなのでこのまま入れてよいと思います。

@beru
Copy link
Contributor Author

beru commented Dec 2, 2018

Review ありがとうございます。Merge します。問題が見つかった場合には後で修正PRを作成します。

@beru beru merged commit 86544be into sakura-editor:master Dec 2, 2018
@m-tmatma m-tmatma added this to the next release milestone Dec 3, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…spRuler

縦スクロール時に不必要にルーラーの再描画がされないように対策
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants