Actions
バグ #1413
closedMiraclePainterの更新のたびに、とくに使われないバックトレース情報が収集されている
プラグイン名:
クラッシュする:
No
Description
実害はそれほどないのですが、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
単なるデッドコードなので消していいと思う
Updated by Shibafu Midorino almost 5 years ago
- Status changed from 分類待ち to パッチ適用待ち
- ブランチ set to topic/1413-eliminate-deadcode-in-mp
Actions