バグ #1273
closed
puts({trim: trim, target: target}) のデバッグメッセージが残っている
Description
#1239 で報告されている問題の (3) について、検索しづらいので別チケットで入れておきます。
mikutterを--debugで実行しているとたまに「{:trim=>"アカウント名", :target=>"@アカウント名"}」という出力があり
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/master/entry/core/plugin/twitter/twitter.rb#L476
で出力していることまで確認したのですが、これが無限に出力され続ける事象を確認しました。
(3) (1)に加え、メンションの後にリンクがあると、score_by_regexpが実行され続けて大量のputs({trim: trim, target: target})が出力されるようになる
これは誤ってコミットしてしまったものですね。削除しておきます。
当該ソース行
source:core/plugin/twitter/twitter.rb@0c877f34#L530
報告事例
https://twitter.com/syuu1228/status/1014611231397707776
mikutter禄に眺めてないけどずっと実行してたらコンソールにトランプおじさんいっぱい出してきた、これはずるい https://t.co/f9ZnlwwMjv
Files
Related issues
Updated by Izumi Tsutsui over 6 years ago
- Related to バグ #1239: 絵文字のあとにメンションがあるとリンクが正しく作らず、リンクもある場合大量のscore_by_regexpが実行され続ける added
Updated by Izumi Tsutsui about 6 years ago
- File 0001-refs-1273.patch 0001-refs-1273.patch added
git format-patch だとログが subject になる（あとメールアドレス大公開になる）んですが、
オススメのパッチの出し方ってどういう方法なんですかね
Updated by Izumi Tsutsui about 6 years ago
- Status changed from 実装待ち to パッチ適用待ち
とりあえずクラッシュしたりはしないのでパッチ適用待ちに変えておきます
Updated by あひる 家鴨 about 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- ブランチ set to topic/1273-remove-debug-message
ブランチにプッシュしました。
確認お願いします。
Updated by Izumi Tsutsui about 6 years ago
- Assignee set to Izumi Tsutsui
- 再現手順 updated (diff)
Updated by Izumi Tsutsui about 6 years ago
- 再現手順 updated (diff)
書く場所を間違えた（わはは
レビュー待ち変更時は担当者も設定して欲しいなり〜
（すぐに試せないのでとりあえず変更）
Updated by Izumi Tsutsui about 6 years ago
- Status changed from レビュー待ち to マージ待ち
とりあえず、topicブランチをマージした手元ブランチで試してみて
落ちたりはせず当該デバッグメッセージも出ていないのでOKと思います
Updated by