Skip to content

Conversation

@xtne6f
Copy link

@xtne6f xtne6f commented Jan 21, 2021

CreateSolidBrush()で作られたブラシが漏れているようです。
タスクマネージャーの「GDIオブジェクト」の項目をみると、EpgDataCap_Bonのそれが1秒に数個ずつ増え続けるはずです。
今のWindowsならシステムを巻き込むことはないと思いますが、アプリが最大確保できるGDIオブジェクトは通常10000個なので、ウィンドウ表示状態なら1時間ほどでEpgDataCap_Bonがクラッシュするか、挙動がおかしくなると思います。

「EpgDataCap_Bon のウインドウの背景色を白に変更した」( 2de0f7a )の修正
@tsukumijima
Copy link
Owner

すみません!今更気づきました…

@tsukumijima
Copy link
Owner

tsukumijima commented Feb 21, 2021

C++ は全く分からず、ネットで調べた知識を元にとりあえず動いているから大丈夫かと思いそのままにしてしまっていました… お恥ずかしい限りです。
最近 EpgDataCap_Bon.exe を開いたまましばらく直で録画させていたことがありましたが、録画自体は動作するもののウインドウ描画がおかしくなってしまったことがありました。今考えればそれが原因だったのだと思います。
すぐに更新したいと思います。わざわざ修正までしていただき本当にありがとうございました!(申し訳ありませんでした…)

@tsukumijima tsukumijima merged commit 6c05843 into tsukumijima:my-build Feb 21, 2021
@xtne6f
Copy link
Author

xtne6f commented Feb 21, 2021

ありがとうございます。
こちらもCreateSolidBrush()1万回ぶん回して観測したかぎりでは、描画の異常のみで済む感じです(使用するBonDriver次第ではわかりませんが)。
多くの人はEpgDataCap_Bon.exeを最小化なりタスクトレイ格納なりで非表示状態にしていると思うので、非表示では再描画されないため、実のところそんなに影響大きくない気がします。何かのついでに更新でも問題ないかと思います。

@tsukumijima
Copy link
Owner

EpgDataCap_Bon.exe を手動で起動すること自体があまりないので、確かにそこまで影響は大きくないかもしれませんね。
久々に upstream の変更を確認したらフォントや UI 周りの大きめの変更があり、コミットログもぐちゃぐちゃになってしまっていたので以前のブランチは一旦放棄して新しくやり直しているところです。

@xtne6f xtne6f deleted the pr-fix branch February 24, 2021 13:19
tsukumijima added a commit that referenced this pull request Mar 6, 2021
旧ブランチの方で xtne6f 氏が #1 にて指摘してくださった箇所を参考に実装
tsukumijima pushed a commit that referenced this pull request Sep 29, 2023
設定保存などが短時間で連続するとロゴ読み込みが無限ループになることがあるのを修正
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.

2 participants