バグ #1515
完了
機能 #887: gtk3
gtk3: mui 関連の deprecated 警告
説明
twitter plugin の gtk3 対応をやっていると
それらからしか呼ばれてなさそうなところでちらほら deprecated の warning が出ていて、
本題のデバッグ中にメッセージが出てめんどくさいのでとりあえず修正したい、という案件です。
パッチは追って。
その1
source:core/mui/gtk_extension.rb@0b99c704#L246
warning: /usr/pkg/lib/ruby/gems/2.7.0/gems/glib2-3.4.9/lib/glib2/deprecatable.rb:177:in `block in __define_deprecated_method__': {MIKUTTER_DIR}/core/mui/gtk_extension.rb:246:in `get_range': 'Gtk::TextBuffer#get_iter_at_offset' has been deprecated. Use 'Gtk::TextBuffer#get_iter_at(:offset => nil)'.
その2
source:core/mui/gtk_inneruserlist.rb@0b99c704#L20
warning: /usr/pkg/lib/ruby/gems/2.7.0/gems/glib2-3.4.9/lib/glib2/deprecatable.rb:123:in `const_missing': {MIKUTTER_DIR}/core/mui/gtk_inneruserlist.rb:20:in `initialize': 'Gtk::SORT_DESCENDING' has been deprecated. Use 'Gtk::SortType::DESCENDING' or ':descending'.
その3
source:core/mui/gtk_userlist.rb@0b99c704#L28
warning: /usr/pkg/lib/ruby/gems/2.7.0/gems/glib2-3.4.9/lib/glib2/deprecatable.rb:177:in `block in __define_deprecated_method__': {MIKUTTER_DIR}/core/mui/gtk_userlist.rb:28:in `initialize': 'Gtk::VScrollbar.new' has been deprecated. Use 'Gtk::Scrollbar.new(:vertical, adjustment)'.
warning: /usr/pkg/lib/ruby/2.7.0/rubygems/deprecate.rb:62:in `block (2 levels) in deprecate': NOTE: Gtk::Box#closeup is deprecated with no replacement. It will be removed on or after 2018-09-01.
Gtk::Box#closeup called from {MIKUTTER_DIR}/core/mui/gtk_userlist.rb:29.
source:core/mui/gtk_extension.rb@0b99c704#L235 の Gtk::Box#closeup を deprecated にするのか?
というところは議論の余地があるかもしれませんが、
手元ではだいぶ消してしまったので修正する方針のままでいいかなあ、と思っています。
ファイル
関連するチケット
Izumi Tsutsui さんが2年以上前に更新
雑diff
toshi_a 初音 さんが2年以上前に更新
- ステータス を 分類待ち から パッチ適用待ち に変更
- 担当者 を Izumi Tsutsui にセット
ありがとうございます。影響少ないので先に取り込んじゃいましょう。
closeupはどうでもいいんですが（残しときたいという思いも薄いし、消したいとも思わない）、みんなもう対応しちゃってそうなので、mikutter本体では利用やめましょうか。
でもあんまりサードパーティープラグインでこの警告出まくるようならdeprecated外したほうが良いかも。対応個数結構多いので、issueは作っときます。
toshi_a 初音 さんが2年以上前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- ブランチ を topic/1515-mui-avoid-deprecated にセット
Izumi Tsutsui さんが2年以上前に更新
- ステータス を レビュー待ち から マージ待ち に変更
toshi_a 初音 さんは #note-2 で書きました:
ありがとうございます。影響少ないので先に取り込んじゃいましょう。
差分としてはOKです。
が、こっちのチケットが先だったせいで
source:plugin/mastodon_account_viewer/relational_container.rb の HBox の修正は
#1520 で当てちゃってました。これだけ手動コンフリクト解決必要です。
closeupはどうでもいいんですが（残しときたいという思いも薄いし、消したいとも思わない）、みんなもう対応しちゃってそうなので、mikutter本体では利用やめましょうか。
でもあんまりサードパーティープラグインでこの警告出まくるようならdeprecated外したほうが良いかも。対応個数結構多いので、issueは作っときます。
本体側で消す方針なら作業は機械的にできるので、プラグイン的にもそういう方針が決まればやりやすいかなとは思います。
個人的に読む側の立場としては、単なる alias 的なやつは探すのにちょっと手間がかかるのでリファレンスで見つかるほうがいい、
というのはあるかもしれません。（GTK3は公式ドキュメントもいまいちですが……）