Project

General

Profile

Actions

バグ #305

closed

古いアイコンキャッシュファイルが30日経過後も削除されない

Added by Izumi Tsutsui about 11 years ago. Updated about 11 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
ブランチ:
クラッシュする:

Description

r281 の変更以降 ~/.mikutter/icons/ 以下にキャッシュされたアイコンファイルは
30日以上古いものは削除される仕様であるが、実際には削除されていないように見える。

core/mui/gtk_webicon.rb の動作を確認してみると、 garbage_collect
呼ばれているものの30日経過後のファイルの検出が正しくされていない
(が rescue nil しているので何も出ない?)ような感じ。

notice を入れまくって延々と動作を見てみると
  • garbage_collect 内の #{icon_dir}#{icondir} の間違い?
  • #{icondir}ICONDIR を返す(~を含む場合がある)ので File.expand_path しないといけない?

の2点を添付パッチのように直すといけてるっぽいです。


Files

gtk_webicon.rb.diff (549 Bytes) gtk_webicon.rb.diff garbage_collect 修正確認用 適当パッチ Izumi Tsutsui, 2011-09-01 18:41
gtk_webicon.rb.diff (634 Bytes) gtk_webicon.rb.diff ファイルの削除は確認した Katsuyoshi MATSUMOTO, 2011-09-22 22:44
Actions #1

Updated by toshi_a 初音 about 11 years ago

不具合確認しました。
このパッチで指摘してもらった点は確認できましたがこれだけでは治らないようなので、調査します

Actions #2

Updated by Izumi Tsutsui about 11 years ago

そもそも garbage_collect ってどのタイミングで呼ばれるはずなんでしょう。
ログ見てると起動してかなり時間経過しても全然呼ばれてない感じが。

Actions #3

Updated by Katsuyoshi MATSUMOTO about 11 years ago

Tsutuiさんのパッチを参考にパッチをつくってみたんだが,
ファイルは削除できるようにはなったはず.

Actions #4

Updated by toshi_a 初音 about 11 years ago

  • Status changed from 新規 to 実装待ち

trunkに取り込んでみました。テストした結果大丈夫そうなので、ちょっと様子を見て0.0.3にも取り込みます

Actions #5

Updated by toshi_a 初音 about 11 years ago

0.0.3にも取り込みます。
ただ、30日では異常にアイコンがたまるので、キャッシュの削除頻度を上げるとか、ある程度の頻度でアクセスのあるアイコンだけキャッシュするとかしてもよさそうとは思った。

Actions #6

Updated by Anonymous about 11 years ago

  • Status changed from 実装待ち to 終了
  • % Done changed from 0 to 100

更新履歴 r516 で適用されました。

Actions

Also available in: Atom PDF