プロジェクト

全般

プロフィール

バグ #1413

完了

MiraclePainterの更新のたびに、とくに使われないバックトレース情報が収集されている

Shibafu Midorino さんが4年以上前に追加. 4年以上前に更新.

ステータス:
終了
優先度:
低め
担当者:
-
対象バージョン:
プラグイン名:
クラッシュする:
いいえ

説明

実害はそれほどないのですが、MiraclePainterの更新のたびにバックトレース情報がインスタンス変数に書き込まれていて、なんか邪魔だなあと思ったので起票します。

shibafu 2:14 AM
なんとなくメモリを眺めてたら変なスタックトレースみたいなStringが大量にあって、なんじゃこりゃあと思ったのでメモっておく。
1. どこで作られてそうか
=> cairo_miracle_painter.rb / on_modify の @modify_source = ... っぽそう
2. 何で使われていそうか
=> 現在のコードでは使われていない。commit 3948158832065f52ad8ec9c65a9b9b4b484ac405 で使われていたようだ。
3. これを消すメリット
=> MiraclePainterを描画するたびに要らんものを作らなくて済む、気持ちの問題っぽい気がする (これによってガメていたメモリがどれくらい空くとか、そういう話に持っていけるほど真剣には見てない)

toshi_a 8:46 AM
単なるデッドコードなので消していいと思う

Shibafu Midorino さんが4年以上前に更新

  • ステータス分類待ち から パッチ適用待ち に変更
  • ブランチtopic/1413-eliminate-deadcode-in-mp にセット

toshi_a 初音 さんが4年以上前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更

toshi_a 初音 さんが4年以上前に更新

  • ステータスレビュー待ち から マージ待ち に変更

OKです、mergeしちゃってください

Shibafu Midorino さんが4年以上前に更新

  • ステータスマージ待ち から 終了 に変更

マージしました。

他の形式にエクスポート: Atom PDF