スパムが酷いため休止中です。移転先検討中です。(2014/8/17)
  新規投稿 ┃ツリー表示 ┃スレッド表示 ┃一覧表示 ┃トピック表示 ┃番号順表示 ┃検索 ┃設定 ┃ホーム  
3 / 437 ツリー ←次へ | 前へ→

DiskInfoDlgMenu.cppにおけるIRegisteredTaskの取り扱いについて もろへ 14/6/18(水) 17:20

Re:DiskInfoDlgMenu.cppにおけるIRegisteredTaskの取... ひよひよ 14/6/18(水) 23:01
優先度 Nano 14/8/7(木) 11:28
Re:優先度 ひよひよ 14/8/7(木) 23:24

Re:DiskInfoDlgMenu.cppにおけるIRegisteredTaskの...
←back ↑menu ↑top forward→
 ひよひよ  - 14/6/18(水) 23:01 -

引用なし
パスワード
   ▼もろへさん:
>まずはじめに、良いソフトウェアをありがとうございます。
>ここでバグ?報告が適切かどうかは分かりませんが、
>
>CrystalDiskInfo6_1_14Srcのソースコードを見ていたところ、
>DiskInfoDlgMenu.cppのBOOL CDiskInfoDlg::RegisterStartup()関数内でのIRegisteredTaskの取り扱いについてですが、909行目で取得し、
>pRootFolder->GetTask(_bstr_t(wszTaskName), &pRegisteredTask);
>(今気付きましたが、ここと下のget_DefinitionでHRESULT取り忘れてますね)
>
>これをそのまま920行目で使いまわしておりますが、
>hr = pRootFolder->RegisterTaskDefinition
>
>MSDNによりますと
>http://msdn.microsoft.com/ja-jp/library/windows/desktop/aa381365%28v=vs.85%29.aspx
>>ppTask [out]
>>An IRegisteredTask interface that represents the new task.
>>Pass in a reference to a NULL IRegisteredTask interface pointer. Referencing a non-NULL pointer can cause a memory leak because the pointer will be overwritten.
>とあり、この状態ではメモリーリークが発生する可能性があるかと思います。
>対応としては、これの前にSAFE_RELEASEしてやるといいとは思います。
ご指摘、解決策のご提案ありがとうございます。
まさにその通りですので次回リリース時に修正させていただきます。

ついにで、タスクスケジューラーに登録する際のプロセス優先度が低い問題も修正しておきました。
pSettings->put_Priority(NORMAL_PRIORITY_CLASS);
を追加。

優先度
←back ↑menu ↑top forward→
 Nano WEB  - 14/8/7(木) 11:28 -

引用なし
パスワード
   多くのPCにCDIを入れて使っていますが、低優先度のほうが個人的にはよかったような・・・

特にネットブックの場合、起動時に大量のサービスが走るので、CDIは後でゆっくりと起動してもらったほうがディスクが傷つきにくいです。
(WaitTimeAtStartup があるから大丈夫なのかな)

Re:優先度
←back ↑menu ↑top forward→
 ひよひよ  - 14/8/7(木) 23:24 -

引用なし
パスワード
   ▼Nanoさん:
>多くのPCにCDIを入れて使っていますが、低優先度のほうが個人的にはよかったような・・・
>
>特にネットブックの場合、起動時に大量のサービスが走るので、CDIは後でゆっくりと起動してもらったほうがディスクが傷つきにくいです。
>(WaitTimeAtStartup があるから大丈夫なのかな)
ここでいう優先度はアプリケーションのプロセス優先度のことなので、起動順番などには関係ありません。今まではスタートアップ設定で起動した場合普通のアプリよりも優先度が低かったのですが、今後は普通のアプリと同じ普通の優先度となります。

3 / 437 ツリー ←次へ | 前へ→
  新規投稿 ┃ツリー表示 ┃スレッド表示 ┃一覧表示 ┃トピック表示 ┃番号順表示 ┃検索 ┃設定 ┃ホーム  
ページ:  ┃  記事番号:   
205617
(SS)C-BOARD v3.8(とほほ改ver2.1) is Free