Project

General

Profile

Actions

バグ #1413

closed

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

Added by Shibafu Midorino almost 5 years ago. Updated almost 5 years ago.

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

Actions #1

Updated by Shibafu Midorino almost 5 years ago

  • Status changed from 分類待ち to パッチ適用待ち
  • ブランチ set to topic/1413-eliminate-deadcode-in-mp
Actions #2

Updated by toshi_a 初音 almost 5 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
Actions #3

Updated by toshi_a 初音 almost 5 years ago

  • Status changed from レビュー待ち to マージ待ち

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

Actions #4

Updated by Shibafu Midorino almost 5 years ago

  • Status changed from マージ待ち to 終了

マージしました。

Actions

Also available in: Atom PDF