プロジェクト

全般

プロフィール

バグ #1273

puts({trim: trim, target: target}) のデバッグメッセージが残っている

Izumi Tsutsui5ヶ月前に追加. 約2ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
ブランチ:
topic/1273-remove-debug-message
クラッシュする:
いいえ

説明

#1239 で報告されている問題の (3) について、検索しづらいので別チケットで入れておきます。

mikutterを--debugで実行しているとたまに「{:trim=>"アカウント名", :target=>"@アカウント名"}」という出力があり
https://dev.mikutter.hachune.net/projects/mikutter/repository/revisions/master/entry/core/plugin/twitter/twitter.rb#L476
で出力していることまで確認したのですが、これが無限に出力され続ける事象を確認しました。

#1239-2

(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

0001-refs-1273.patch (1.02 KB) 0001-refs-1273.patch Izumi Tsutsui, 2018-09-09 12:53

関連するチケット

関連している バグ #1239: 絵文字のあとにメンションがあるとリンクが正しく作らず、リンクもある場合大量のscore_by_regexpが実行され続ける終了

関係しているリビジョン

リビジョン 1ff37e5f (差分)
Izumi Tsutsui約2ヶ月前に追加

不要なデバッグメッセージを削除 refs #1273

履歴

#1 Izumi Tsutsui5ヶ月前に更新

  • 関連している バグ #1239: 絵文字のあとにメンションがあるとリンクが正しく作らず、リンクもある場合大量のscore_by_regexpが実行され続ける を追加

#2 Izumi Tsutsui2ヶ月前に更新

git format-patch だとログが subject になる(あとメールアドレス大公開になる)んですが、
オススメのパッチの出し方ってどういう方法なんですかね

#3 Izumi Tsutsui2ヶ月前に更新

  • ステータス実装待ち から パッチ適用待ち に変更

とりあえずクラッシュしたりはしないのでパッチ適用待ちに変えておきます

#4 あひる 家鴨約2ヶ月前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • ブランチtopic/1273-remove-debug-message にセット

ブランチにプッシュしました。
確認お願いします。

#5 Izumi Tsutsui約2ヶ月前に更新

  • 担当者Izumi Tsutsui にセット
  • 再現手順 を更新 (diff)

#6 Izumi Tsutsui約2ヶ月前に更新

  • 再現手順 を更新 (diff)

書く場所を間違えた(わはは


レビュー待ち変更時は担当者も設定して欲しいなり〜
(すぐに試せないのでとりあえず変更)

#7 Izumi Tsutsui約2ヶ月前に更新

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

とりあえず、topicブランチをマージした手元ブランチで試してみて
落ちたりはせず当該デバッグメッセージも出ていないのでOKと思います

#8 cob odo約2ヶ月前に更新

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

masterにmergeしました

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