Project

General

Profile

バグ #1273

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

Added by Izumi Tsutsui about 1 year ago. Updated 10 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
topic/1273-remove-debug-message
クラッシュする:
No

Description

#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


Files

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

Related issues

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

Associated revisions

Revision 1ff37e5f (diff)
Added by Izumi Tsutsui 10 months ago

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

History

#1

Updated by Izumi Tsutsui about 1 year ago

  • Related to バグ #1239: 絵文字のあとにメンションがあるとリンクが正しく作らず、リンクもある場合大量のscore_by_regexpが実行され続ける added
#2

Updated by Izumi Tsutsui 11 months ago

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

#3

Updated by Izumi Tsutsui 11 months ago

  • Status changed from 実装待ち to パッチ適用待ち

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

#4

Updated by あひる 家鴨 10 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • ブランチ set to topic/1273-remove-debug-message

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

#5

Updated by Izumi Tsutsui 10 months ago

  • Assignee set to Izumi Tsutsui
  • 再現手順 updated (diff)
#6

Updated by Izumi Tsutsui 10 months ago

  • 再現手順 updated (diff)

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


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

#7

Updated by Izumi Tsutsui 10 months ago

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

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

#8

Updated by cob odo 10 months ago

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

masterにmergeしました

Also available in: Atom PDF