Project

General

Profile

バグ #1413

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

Added by Shibafu Midorino 9 months ago. Updated 9 months ago.

Status:
終了
Priority:
低め
Assignee:
-
Target version:
プラグイン名:
ブランチ:
topic/1413-eliminate-deadcode-in-mp
クラッシュする:
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
単なるデッドコードなので消していいと思う

Also available in: Atom PDF