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

CEditView::CreateOrUpdateCompatibleBitmap において画面バッファを作成する条件を変更 #1015

Merged
merged 1 commit into from
Aug 25, 2019
Merged

CEditView::CreateOrUpdateCompatibleBitmap において画面バッファを作成する条件を変更 #1015

merged 1 commit into from
Aug 25, 2019

Conversation

beru
Copy link
Contributor

@beru beru commented Aug 25, 2019

PR の目的

ウィンドウサイズの変更時に画面バッファの再作成処理が頻繁に動作しないようにする。

カテゴリ

  • 速度向上

PR の背景

新規に作成する画面バッファの縦幅と横幅を既に作成済みの画面バッファの縦幅と横幅と比較して大きい場合のみ作成するように条件を変更する。

新規に作成する画面バッファの縦幅横幅ともに小さいならば、既に作成済みの画面バッファをわざわざ破棄して小さい画面バッファを作成する事はせずに使いまわすようにする。

PR のメリット

ウィンドウサイズの変更時に画面バッファの再作成処理が頻繁に動作しないようになり、処理負担が減ります。

PR のデメリット (トレードオフとかあれば)

ウィンドウサイズを小さくしても画面バッファの再作成を行わないため、メモリ使用量が減りません。

ただし画面バッファのメモリ使用量はそこまで大きいものでは無い為、実質的に問題無いと思います。

PR の影響範囲

画面バッファを扱う箇所

新規に作成する画面バッファの縦幅と横幅を既に作成済みの画面バッファの縦幅と横幅と比較して大きい場合のみ作成するように条件を変更
新規に作成する画面バッファの縦幅横幅ともに小さいならば、既に作成済みの画面バッファを破棄して小さい画面バッファを作成する事はせずに使いまわすようにする。
画面サイズの変更時に画面バッファの再作成処理が頻繁に動作しないようにする。
@beru beru added the 🚅 speed up 🚀 高速化 label Aug 25, 2019
@AppVeyorBot
Copy link

Build sakura 1.0.2191 completed (commit 8052aeca10 by @)

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 Aug 25, 2019

@berryzplus さん
レビューありがとうございます。
それでは Merge します。もし問題が見つかった場合は別PRで対処します。

@beru beru merged commit d1fb8e1 into sakura-editor:master Aug 25, 2019
@beru beru deleted the CEditView__CreateOrUpdateCompatibleBitmap branch August 25, 2019 08:55
@m-tmatma m-tmatma added this to the v2.4.0 milestone Dec 29, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
…dateCompatibleBitmap

CEditView::CreateOrUpdateCompatibleBitmap において画面バッファを作成する条件を変更
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.

4 participants