バグ #1515
closed
機能 #887: gtk3
gtk3: mui 関連の deprecated 警告
Description
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 にするのか?
というところは議論の余地があるかもしれませんが、
手元ではだいぶ消してしまったので修正する方針のままでいいかなあ、と思っています。
Files
Updated by Izumi Tsutsui over 3 years ago
雑diff
Updated by toshi_a 初音 over 3 years ago
- Status changed from 分類待ち to パッチ適用待ち
- Assignee set to Izumi Tsutsui
ありがとうございます。影響少ないので先に取り込んじゃいましょう。
closeupはどうでもいいんですが（残しときたいという思いも薄いし、消したいとも思わない）、みんなもう対応しちゃってそうなので、mikutter本体では利用やめましょうか。
でもあんまりサードパーティープラグインでこの警告出まくるようならdeprecated外したほうが良いかも。対応個数結構多いので、issueは作っときます。
Updated by toshi_a 初音 over 3 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- ブランチ set to topic/1515-mui-avoid-deprecated
Updated by Izumi Tsutsui over 3 years ago
- Status changed from レビュー待ち to マージ待ち
toshi_a 初音 さんは #note-2 で書きました:
ありがとうございます。影響少ないので先に取り込んじゃいましょう。
差分としてはOKです。
が、こっちのチケットが先だったせいで
source:plugin/mastodon_account_viewer/relational_container.rb の HBox の修正は
#1520 で当てちゃってました。これだけ手動コンフリクト解決必要です。
closeupはどうでもいいんですが（残しときたいという思いも薄いし、消したいとも思わない）、みんなもう対応しちゃってそうなので、mikutter本体では利用やめましょうか。
でもあんまりサードパーティープラグインでこの警告出まくるようならdeprecated外したほうが良いかも。対応個数結構多いので、issueは作っときます。
本体側で消す方針なら作業は機械的にできるので、プラグイン的にもそういう方針が決まればやりやすいかなとは思います。
個人的に読む側の立場としては、単なる alias 的なやつは探すのにちょっと手間がかかるのでリファレンスで見つかるほうがいい、
というのはあるかもしれません。（GTK3は公式ドキュメントもいまいちですが……）
Updated by Izumi Tsutsui over 3 years ago
- Related to 提案 #1525: closeupメソッドの利用をやめる added